[PATCH] D19820: Enable loopreroll for sext of loop control only IV

James Molloy via llvm-commits llvm-commits at lists.llvm.org
Wed May 4 06:50:49 PDT 2016


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

Hi,

This looks generally fine to me. I've got one comment.

Cheers,

James


================
Comment at: lib/Transforms/Scalar/LoopRerollPass.cpp:561
@@ +560,3 @@
+                isa<SExtInst>(UUser))
+              UUser = dyn_cast<Instruction>(*(UUser->user_begin()));
+            if (!isCompareUsedByBranch(dyn_cast<Instruction>(UUser)))
----------------
You shouldn't need the cast here.


Repository:
  rL LLVM

http://reviews.llvm.org/D19820





More information about the llvm-commits mailing list