tree 7a92f028afab9da9ce9eb77be5362c6bf2f0a5b9
parent 1d00b505de74455fc14175d2d3decdad180293f4
author Nirvedh Meshram <96096277+nirvedhmeshram@users.noreply.github.com> 1713214494 -0500
committer GitHub <noreply@github.com> 1713214494 -0500
gpgsig -----BEGIN PGP SIGNATURE-----
 
 wsFcBAABCAAQBQJmHZQeCRC1aQ7uu5UhlAAA2x4QAFHm3MOGGen9kUXCAkwkaxMU
 GFGRg6yrEypaAuTxhnwhVpUcxvVIWRNDT736erDy/4nxaEw7Rdmr/ZR9AbyjcnDY
 3gP8s8Te1hQZNdS+g2YGNDJqWCJLwqqd5t2ezDDjgyeXDCd2UDqnZzLUE6hf1Bbr
 mAjXQazS+BTOvOIjsMqfWyhq7yjoflcJAfdLZ58+pq+ege2nPSNS90FbKovzzTFS
 TWg6YEix/hckTsQENdwLm5T/DugSjb2dQlBLb8sNUhcsWoYI+XDbaBJTZZfr39sF
 U9J4akDdZtJbOcjBVrptaE6DNQrqNUW9eWmxZmw/5W4rxMXKK3K72oEKgKaT5AZ3
 WQReu6bE6RI3jKmwbrwdGNA110MekZRXUn83LdN32lNPw1NwuZqnxr/EODlXDkUX
 7u4RubiOBXjBMkzs/JjtPjVoZJgSygAjkZOrYqBjBjEIcvtDn5BhKECpVa0BjNRX
 c1ZDGiOajZSogkC52XVTCEQMCW0dZtLRXMOnz8JtVZeFFVuMKAGDpN9m9SNDoUxP
 mqt6kR030izMDGfu+axAjLbluPNX+nGayNfsgXuhIRavC+qVEH/XrdzApXDxFFmG
 uayZv29bpvTyxgwbGNP4a//wkhHU4cHuyEETLc7s980RsNK+vYlm5xnf79HUKP5R
 RIdeor0N1SDuH1s5jmL8
 =9+dg
 -----END PGP SIGNATURE-----
 

Add ability to call the same custom dispatch multiple times when using pdl patterns. (#16967)

Following changes were made to allow multiple calls
1. In the rewriteAsFlowDispatch function we now check if the executable
being created already exists and if it does we call that executable
rather than creating a new one.
2. Adds a naming scheme to make unique executable names based on
input/output types.
3. It was discovered that the created executable was not handling
offsets correctly, the reason was the folder on
memref.extract_stided_metadata op which would break the link between the
offset and the function call before the offset was set. Further it was
also noticed that the memref type did not have a MemRefLayoutAttr so it
implicitly assumed zero offset. Now we add it to the memref type and
assume the offset is dynamic at the time of executable creation so it
wont get folded even with the incorrect folder from upstream. To further
safeguard in case the offset gets optimized to zero and later changed we
add a iree_codegen.extract_strided_metadata which is an extact replica
of the memref dialect op with the exception that it does not have the
incorrect folder.
4. Lastly we add a trivial pattern to convert
iree_codegen.extract_strided_metadata to memref.extract_strided_metadata
in IREEExpandStridedMetadata pass. It should be noted that the
expectation is that the memref type should not say it has zero offset
here if we dont have a zero offset. If the type says it has zero or any
static offset, the upstream folder will still trigger and break the link
to any consumers before our
ResolveExtractMetadataFromHalInterfaceBindingSubspan pattern can resolve
it.