[PATCH] D110822: [GVN] Simple GVN hoist - loads and stores

[Arthur Eubanks via Phabricator via llvm-commits]

aeubanks added a comment.

In D110822#3099586 <https://reviews.llvm.org/D110822#3099586>, @mkazantsev wrote:

> (Please don't mind -1 for me as it was just for sorting out review list purposes)
>
> The general question I have - we are planning to transit from GVN to NewGVN, or at least so I heard. Is NewGVN already able to handle that? Is there a further plan to port these changes there?

I believe transitioning to NewGVN is still not definitive, there are a lot of things it doesn't support yet and it has bugs.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D110822/new/

https://reviews.llvm.org/D110822