[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