[PATCH] D12285: [LV] Switch to using canonical induction variables.

Adam Nemet via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 25 20:52:53 PDT 2015


anemet added reviewers: hfinkel, aschwaighofer, nadav.
anemet added a comment.

I am OK with this but I'd like someone else to sign off on this one too.  Alternatively showing that there is no relevant asm diff on the testsuite (including externals) is probably also a good indication that this case is over-engineered locally.

When you say InstCombine do you mean LSR?

We should probably also add a comment that OldInduction is now a canonical induction variable.  And remove ExtendedIdx, which I think is unused now.


Repository:
  rL LLVM

http://reviews.llvm.org/D12285





More information about the llvm-commits mailing list