[PATCH] D43865: [NewGVN] Create new ValuePHI node, if the number of operands does not match.

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 19 07:38:29 PDT 2018


fhahn added a comment.

In https://reviews.llvm.org/D43865#1072149, @dberlin wrote:

> So, just to put this on the review: the current code on this review definitely not the right fix  compared to the patch i sent :)
>  I think we should commit that split/cleanup (not sure if you did it as part of the last patch.
>  It sounds like you had time to bootstrap/test it.  If so, please feel free to commit it


Yep, the diff in Phab is out-dated. I'll commit https://reviews.llvm.org/D42180 first, and then rebase your patch and do another bootstrap round and update the diff here.


https://reviews.llvm.org/D43865





More information about the llvm-commits mailing list