[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