[PATCH] D116496: [unroll] Prune all but first copy of invariant exit
Nikita Popov via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Jan 3 01:11:49 PST 2022
nikic accepted this revision.
nikic added a comment.
This revision is now accepted and ready to land.
LG apart from the previous note on tests. I think unrolling is an area where it is worthwhile to do cleanup as part of the transform (as long as it remains simple), because we may be producing a large amount of code, and this reduces the amount of work follow-on transforms have to do. Additionally, it is worth noting that runtime unrolling is performed very late in the pipeline, and we generally cannot expect any non-trivial redundancies exposed by that to be optimized away.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D116496/new/
https://reviews.llvm.org/D116496
More information about the llvm-commits
mailing list