[PATCH] D23075: [IndVarSimplify] Extend trip count instead of truncating IV in LFTR, when original IV does not overflow

Sanjoy Das via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 8 10:45:09 PDT 2016


sanjoy added inline comments.

================
Comment at: test/Transforms/IndVarSimplify/lftr-wide-trip-count.ll:81
@@ +80,3 @@
+; br i1 %exitcond
+}
+
----------------
Wherever pertinent, can you also please check the full `icmp ne` expression (that is, not just the prefix)?  Since that's the whole change basically (changing what `icmp` expression we generate) I think we should be rigorous there.


https://reviews.llvm.org/D23075





More information about the llvm-commits mailing list