[PATCH] D32252: [GVN] Add phi-translate for scalarpre as a temporary solution
Daniel Berlin via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue May 23 20:38:53 PDT 2017
dberlin accepted this revision.
dberlin added a comment.
At this point, i think this is reasonable. we know what the cost is, it seems reasonable given the benefits, and we know it's not long term.
the code seems clean etc.
i"m going to accept this, but i'd just wait a day to see if anyone else has comments before committing.
Thank you for working through this with folks :)
Repository:
rL LLVM
https://reviews.llvm.org/D32252
More information about the llvm-commits
mailing list