Creating maximally folded and composd affine.apply operation during
FoldMemRefAliasOps composes better with other transformations without having
to interleave canonicalization passes.
Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
Comment Actions
Note the automatic d -> s promotion due to affine scoping rules.
I would personally prefer to trigger the promotion explicitly rather than have it apply everywhere all the time but this has not been a hard blocker recently.
Comment Actions
LGTM
mlir/lib/Dialect/MemRef/Transforms/FoldMemRefAliasOps.cpp | ||
---|---|---|
14–16 | Merge this block of includes with the one below; no need to keep them separate unless this block only has Passes.h. | |
22 | This include isn't needed I think. | |
77 | Include a short code comment here along the lines you have in the commit summary. |
Merge this block of includes with the one below; no need to keep them separate unless this block only has Passes.h.