[PATCH] D18019: [PM] Port GVN to the new pass manager, wire it up, and teach a couple of tests to run GVN in both modes.

Chandler Carruth via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 11 00:55:28 PST 2016


chandlerc added a comment.

In http://reviews.llvm.org/D18019#372148, @dberlin wrote:

> Thanks so much for doing this.
>  This looks completely reasonable to me.  While, as you say, this could be cleaned up further, and needs to be better documented, i don't think that's your job.
>  so LGTM


Thanks for the review! Happy to help with any subsequent cleanups if I can, but yea, I'll probably focus on pass manager-y things for now.


http://reviews.llvm.org/D18019





More information about the llvm-commits mailing list