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

[Nadav Rotem via llvm-commits]

nadav added a comment.

I think that this is a good change. In many other places in the vectorizer the design was that we let other passes (such as CSE, InstCombine and LSR) clean up after us. I am totally okay with letting LSR do the cleanup. If I remember correctly we have always relied on LSR to do the cleanup and I don't remember why we have the logic for searching other induction variables.

LGTM.


Repository:
  rL LLVM

http://reviews.llvm.org/D12285