[PATCH] D12745: [IndVars] Widen more comparisons for non-negative induction vars.
Philip Reames via llvm-commits
llvm-commits at lists.llvm.org
Thu Sep 10 14:46:19 PDT 2015
reames added inline comments.
================
Comment at: test/Transforms/IndVarSimplify/widen-loop-comp.ll:213
@@ +212,3 @@
+for.body:
+ %idxprom = zext i32 %i.0 to i64
+ %arrayidx = getelementptr inbounds i32, i32* %a, i64 %idxprom
----------------
I would prefer you left them. The zero based increasing loop is a simpler, easier to understand, and really important case. I think there's value even if they're covered by the additional cases.
http://reviews.llvm.org/D12745
More information about the llvm-commits
mailing list