[PATCH] D102249: [LoopFlatten] Simplify loops so that the pass can operate on unsimplified loops.

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 12 11:24:36 PDT 2021


fhahn added inline comments.


================
Comment at: llvm/lib/Transforms/Scalar/LoopFlatten.cpp:692
+    Changed |=
+        simplifyLoop(L, DT, LI, SE, AC, nullptr, false /* PreserveLCSSA */);
+    Changed |= formLCSSARecursively(*L, *DT, LI, SE);
----------------
Would it not be more natural to do that directly in the main loop in `Flatten()` rather than duplicating the loop?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D102249



More information about the llvm-commits mailing list