Re: [PATCH] D17268: [LAA] Function 'isStridedPtr' returns additional result “Loop *Lp” via function argument and add appropriate checks out of the 'isStridedPtr'.

[Roman Shirokiy via llvm-commits]

roman.shirokiy updated this revision to Diff 55632.
roman.shirokiy added a comment.

Thanks Hal for the review!

Updated diff according to Hal's corrections. However, extra overload for this function turns up as a too much of repetitive code for the sake of few different lines, but approach with Lp being nullptr seems to be clean and allows the needed change for PR26314.


http://reviews.llvm.org/D17268

Files:
  include/llvm/Analysis/LoopAccessAnalysis.h
  lib/Analysis/LoopAccessAnalysis.cpp
  lib/Transforms/Scalar/LoopLoadElimination.cpp
  lib/Transforms/Vectorize/LoopVectorize.cpp
  test/Analysis/LoopAccessAnalysis/multiple-strides-rt-memory-checks.ll
  test/Transforms/LoopVectorize/multiple-strides-vectorization.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D17268.55632.patch
Type: text/x-patch
Size: 8656 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160429/2eacb1b4/attachment.bin>