[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
Fri Nov 2 07:52:29 PDT 2018


hfinkel added a comment.

> I'm not very familiar with the MemorySSA pass but based on a fairly quick skim of it ...
>  It looks like the MemoryDef/MemoryUse objects are attached to the BB rather than the instructions.

I'll write more later, but quickly, this is incorrect. MemorySSA has per-instruction granularity.


Repository:
  rL LLVM

https://reviews.llvm.org/D53203





More information about the llvm-commits mailing list