[PATCH] D110057: [LoopFlatten] Move it to a LoopPassManager

[Dave Green via Phabricator via llvm-commits]

dmgreen added a comment.

New pass manager loop pass invalidation isn't exactly my area of expertise. My main issue with moving the pass was that performance was worse. I don't now remember why, but I have a vague memory of widening IV's in the flattening pass, if they have already been partially widened in indvarsimplify.

It may be important to note that LoopFlatten isn't a Loop pass. It's a LoopNest Pass and has been since D102904 <https://reviews.llvm.org/D102904>.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D110057/new/

https://reviews.llvm.org/D110057