[PATCH] D36309: [LoopUnroll] Enable option to peel remainder loop

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 4 11:24:29 PDT 2017


efriedma added a comment.

This makes sense.



================
Comment at: lib/Transforms/Utils/LoopUnrollRuntime.cpp:894
+  if (remainderLoop && PeelRemainder)
+    peelLoop(remainderLoop, Count - 1, LI, SE, DT, AC, PreserveLCSSA);
+
----------------
I think I would write this using UnrollLoop: full unrolling will result in a slightly simpler CFG because the backedge goes away.  (Granted, later optimizations will probably clean it up either way.)


https://reviews.llvm.org/D36309





More information about the llvm-commits mailing list