[PATCH] D45320: [MemDep] Fixed handling of invariant.group

Daniel Berlin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 5 13:55:53 PDT 2018


dberlin added a comment.

"I know, I also wish we would get rid of MemDep asap, but as long as it's there, it would be good to avoid weird miscompilations that causes calls to vtable."

But it still will, just not in this particular way. 
But it will still fail in very common ways anyway :)

We may actually have to disable preservation entirely for the moment anyway.


Repository:
  rL LLVM

https://reviews.llvm.org/D45320





More information about the llvm-commits mailing list