[PATCH] D42381: [DA] Correct size parameter from dependency analysis to AA

Hal Finkel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 9 04:08:09 PDT 2018


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

In https://reviews.llvm.org/D42381#1061394, @dmgreen wrote:

> But in here:
>
>   for (int i = 0; i < n; i++) {
>     A[i + 2] = i;
>     ... = A[i];
>
>
> the load and store are mayalias. We need the fact that the underlying obects are mustalias (otherwise a number of tests are failing in a way that looks like it's discovering less dependencies, more are confused)


Ah, okay. LGTM.


https://reviews.llvm.org/D42381





More information about the llvm-commits mailing list