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

Elena Demikhovsky via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 20 02:40:25 PDT 2016


delena added a comment.

> OK, thanks for the explanation but something is still not clear.  If the stride is 3 how does that convert into isConsecutivePtr to return true?  We are checking for Stride == 1 || Stride == -1 in the new version of isConsecutivePtr.


isConsecutivePtr() still returns 0. But it has a side effect. This line: PSE.setNoOverflow(Ptr, SCEVWrapPredicate::IncrementNUSW);
This NoOverlflow flag is set for the pointer (Ptr) and checked later, in another function.


Repository:
  rL LLVM

http://reviews.llvm.org/D20789





More information about the llvm-commits mailing list