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

[Adam Nemet via llvm-commits]

anemet accepted this revision.
anemet added a comment.
This revision is now accepted and ready to land.

LGTM with the improved comments in the testcases.  Thanks!


================
Comment at: test/Analysis/LoopAccessAnalysis/multiple-strides-rt-memory-checks.ll:1-2
@@ +1,3 @@
+; This is the test case from PR26314.
+; RUN: opt -loop-accesses -analyze -S < %s | FileCheck %s
+; CHECK: function 'Test':
----------------
Can you please add a bit more explanation here and in the other testcase.

Please include the C version and a few words like:

```
When we were retrying dependence checking with memchecks only, the loop-invariant access in the inner loop was incorrectly determined to be wrapping because it was not strided in the inner loop.
```


http://reviews.llvm.org/D17268