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

[James Molloy via llvm-commits]

jmolloy updated this revision to Diff 33524.
jmolloy added a comment.

Hi Adam,

Rebased and uploaded with full context.

Would it be acceptable to commit this patch and monitor for performance regressions? Without a specific regression test that shows a pattern being pessimized, and with my own testing showing no regressions, to me it makes sense to proceed and watch performance numbers carefully for a regression.

Is this acceptable to you?

Cheers,

James


Repository:
  rL LLVM

http://reviews.llvm.org/D12285

Files:
  lib/Transforms/Vectorize/LoopVectorize.cpp
  test/Transforms/LoopVectorize/induction.ll
  test/Transforms/LoopVectorize/reverse_induction.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D12285.33524.patch
Type: text/x-patch
Size: 4672 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150830/10393ff4/attachment.bin>