[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