[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 01:34:23 PDT 2018


hfinkel added a comment.

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

> Thanks for the pointers. I've folded the tbaa changes into this patch as they are related. This now checks the alias on the original locations, on the assumption that if they don't alias for any reason then there is no dependency. Then falls back to underlying object.


Does the fallback add anything? I think that the AA query should catch all relevant cases (i.e., you can just return the result of the AA query and be done).


https://reviews.llvm.org/D42381





More information about the llvm-commits mailing list