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

[silviu.baranga@arm.com via llvm-commits]

sbaranga added a comment.

Hi Elena,

It looks like we don't care if the pointer is wrapping in isConsecutivePtr, but getPtrStride is testing it since this is required for LAA?

In that case it might be better to either factor out the wrapping checks from getPtrStride or change the interface so that getPtrStride takes an Assume parameter and a ShouldCheckWrap parameter?

-Silviu


Repository:
  rL LLVM

http://reviews.llvm.org/D20789