[PATCH] D51168: [NewGVN] Make sure we do not add a user to itself.

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Nov 1 09:17:11 PDT 2018


fhahn added a comment.

In https://reviews.llvm.org/D51168#1283117, @efriedma wrote:

> I don't really know the relevant code well enough, and I don't think I'll have time to study it in the near future.
>
> You might want to ask on llvmdev to see if there's anyone interested in reviewing NewGVN patches.


@efriedma thanks for all the reviews you did already for NewGVN related changes, I really appreciate it! Thanks for the suggestion, I guess I could prepare a few additional patches and send some kind of status update with them to llvmdev.

In https://reviews.llvm.org/D51168#1283253, @davide wrote:

> I'll take a look early next week. Sorry, I don't work on scalar optimizations anymore on llvm these days, so I need to page this code in again.


That would be great!


https://reviews.llvm.org/D51168





More information about the llvm-commits mailing list