[PATCH] D19258: Loop vectorization with induction variable with non-constant step.

[Elena Demikhovsky via llvm-commits]

delena added a comment.

Hi Hal,

I checked again. SE->getAddExpr(SE->getSCEV(A), SE->getSCEV(B)) does not always give better results than just createAdd(A, B).
It happens when we mix getAddExpr on SCEVs and ADD/SUB operations. As a result, we receive redundant intermediate values being calculated in different ways and Instcombine is unable to reduce them all.

Could you, please, continue reviewing this patch. Thank you.


Repository:
  rL LLVM

http://reviews.llvm.org/D19258