[PATCH] D17279: [Sink] Don't move calls to readonly functions across stores

Nicolai Hähnle via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 10 11:36:33 PST 2016


nhaehnle added a comment.

In http://reviews.llvm.org/D17279#353238, @dberlin wrote:

> Doesn't AA have an existing API that gives you exactly the info you want
>  here (Does this call ref the memory touched the things in the store list)


I see no such interface in AAResults at least. All queries test $something against a MemoryLocation, but the Stores are instructions because they could also be call sites. There would have to be an interface to test an instruction against an instruction, or two call sites against each other.


http://reviews.llvm.org/D17279





More information about the llvm-commits mailing list