[PATCH] D53203: Allow MemoryLocation to carry pre-existing knowledge to AA to elide expensive repeated checks

Hal Finkel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 31 19:18:05 PDT 2018


hfinkel added a comment.

In https://reviews.llvm.org/D53203#1282454, @dsanders wrote:

> Hi Hal,
>
> I didn't spot you in CODE_OWNERS.txt at first because I was searching for 'Alias' instead of 'alias' :-). Would you be able to take a look at this patch too?


Thanks. I suppose that I should echo Chris's question about the proposed local-dominance cache with respect to this change too: If we switch this pass to using MemorySSA does this problem go away?


Repository:
  rL LLVM

https://reviews.llvm.org/D53203





More information about the llvm-commits mailing list