[PATCH] D90402: [LoopFlatten] Run it earlier, just before IndVarSimplify

Dave Green via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Nov 5 02:18:44 PST 2020


dmgreen added a comment.

OK yeah, they look like they are already in different loop managers on the old pass manager side.



================
Comment at: llvm/lib/Passes/PassBuilder.cpp:527
+  if (EnableLoopFlatten)
+    LPM2.addPass(LoopFlattenPass());
+
----------------
I think this one would be better put into a new LoopPassManager, LPM3, if flattening is enabled. I think that's how it's meant to work.


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

https://reviews.llvm.org/D90402



More information about the llvm-commits mailing list