[PATCH] D116612: [LoopFlatten] Move it from LPM2 to LPM1

Nikita Popov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 4 11:47:59 PST 2022


nikic added a comment.

Could you please add a PhaseOrdering test that demonstrates the issue?



================
Comment at: llvm/lib/Transforms/Scalar/LoopFlatten.cpp:828
+  PA.preserve<MemorySSAAnalysis>();
+  return PA;
 }
----------------
This is very likely not correct without additional changes. This pass changes control flow, and that usually requires updates to MSSA as well.


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

https://reviews.llvm.org/D116612



More information about the llvm-commits mailing list