[PATCH] D18158: Adding ability to unroll loops using epilogue remainder.

Michael Zolotukhin via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 1 15:42:06 PDT 2016


mzolotukhin accepted this revision.
mzolotukhin added a comment.
This revision is now accepted and ready to land.

Thanks, I like it better now! It looks good to me (with one remark inline).

Michael


================
Comment at: lib/Transforms/Utils/LoopUnrollRuntime.cpp:652
@@ +651,3 @@
+    // I should be multiply of Count.
+    IRBuilder<> B2(NewPreHeader->getTerminator());
+    Value *TestVal = B2.CreateSub(TripCount, ModVal, "unroll_iter");
----------------
You could reuse existing `IRBuilder` (just use `SetInsertPoint` method).


http://reviews.llvm.org/D18158





More information about the llvm-commits mailing list