[PATCH] D91525: [LoopLoadElim] Make sure all loops are in simplify form. PR48150

Arthur Eubanks via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Nov 20 09:33:46 PST 2020


aeubanks added inline comments.


================
Comment at: llvm/lib/Transforms/Scalar/LoopLoadElimination.cpp:627
+    for (Loop *L : depth_first(TopLevelLoop)) {
+      Changed |= simplifyLoop(L, &DT, &LI, SE, AC, /*MSSAU*/ nullptr, false);
       // We only handle inner-most loops.
----------------
does every loop need to be simplified? or can the call be moved into the processing of the Worklist below?


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

https://reviews.llvm.org/D91525



More information about the llvm-commits mailing list