[PATCH] D20789: Consecutive memory access in Loop Vectorizer - fixed and simplified

[Elena Demikhovsky via llvm-commits]

delena requested a review of this revision.

================
Comment at: ../lib/Transforms/Vectorize/LoopVectorize.cpp:2677
@@ -2748,6 +2676,3 @@
     } else if (Gep) {
       setDebugLocFromInst(Builder, Gep);
       // The last index does not have to be the induction. It can be
----------------
This change I made after buildbot failed at assertion. The main problem that I wasn't able to reproduce this failure. (llvm self build under sanitizer ..).
I fixed the failure by changing the first "if" from "if-induction" to "if-not-loop-invariant" and the buildbot passed all tests.
Do you think that this fix is correct?
May be you can tell me how to reproduce the failure?
Thanks.


Repository:
  rL LLVM

https://reviews.llvm.org/D20789