[PATCH] D40375: Use MemorySSA in LICM to do sinking and hoisting.

Daniel Berlin via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 19 17:34:27 PST 2017


>
>
> ----------------
> sanjoy wrote:
> > This seems fishy -- if `volatile` loads do not invalidate pointers then
> why are they treated as defs by MSSA?
> Well, because they should be treated as defs.



Doubtful.  We need to respect various things  but they are not defs.
They do not change memory.
As mentioned, we simply don't represent ordering/etc chains separately from
use-def chains right now,.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20171219/9d007a52/attachment.html>


More information about the llvm-commits mailing list