[PATCH] D28137: [Devirtualization] MemDep returns non-local !invariant.group dependencies

Piotr Padlewski via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 11 08:08:38 PST 2017


Prazek added a comment.

In https://reviews.llvm.org/D28137#642786, @dberlin wrote:

> The N^2 loop worries me a lot, because it means i can easily construct cases where you've made memdep N^3 or N^4 in the case of invariant group dependencies.
>
> But i guess we'll see.


Yea, but at least for clang it won't matter because it deosn't use invariant.group as default. I will make sure MemSSA will support it with constant time before turing it default.


https://reviews.llvm.org/D28137





More information about the llvm-commits mailing list