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

[James Molloy via llvm-commits]

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