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

Stelios Ioannou via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 12 11:36:56 PDT 2021


stelios-arm 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);
----------------
fhahn wrote:
> Would it not be more natural to do that directly in the main loop in `Flatten()` rather than duplicating the loop?
Oh yes, I missed that. 


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