[PATCH] D19553: Unroll pass restructure.

Evgeny Stupachenko via llvm-commits llvm-commits at lists.llvm.org
Thu May 5 15:41:39 PDT 2016


evstupac added inline comments.

================
Comment at: lib/Transforms/Scalar/LoopUnrollPass.cpp:540
@@ +539,3 @@
+    UP.AllowExpensiveTripCount = true;
+    UP.Force = true;
+    if (UP.AllowRemainder &&
----------------
Currently the formula have different parameters at different places. I'm planing to unify this, but this will change current behavior. So this is still  a good point, but I'd like to fix this in separate patch.


Repository:
  rL LLVM

http://reviews.llvm.org/D19553





More information about the llvm-commits mailing list