[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