[PATCH] D103362: [LoopUnroll] Separate peeling from unrolling

Philip Reames via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 4 12:27:47 PDT 2021


reames accepted this revision.
reames added a comment.

LGTM



================
Comment at: llvm/lib/Transforms/Scalar/LoopUnrollPass.cpp:1177
+    if (peelLoop(L, PP.PeelCount, LI, &SE, &DT, &AC, PreserveLCSSA)) {
+      simplifyLoopAfterUnroll(L, true, LI, &SE, &DT, &AC, &TTI);
+      // If the loop was peeled, we already "used up" the profile information
----------------
In the future, we really should pull out a function with the other loop simplification, and use that here, but that's for the future.  


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

https://reviews.llvm.org/D103362



More information about the llvm-commits mailing list