[PATCH] D66909: [GVN] Disallow phi translation for expressions with calls.

Thu Aug 29 13:50:50 PDT 2019

wmi added a comment.

In D66909#1651640 <https://reviews.llvm.org/D66909#1651640>, @wmi wrote:

> Thanks Alina to analyze the problem and write the fix.
>
> I agree with your analysis and I believe the patch could fix the problem. But I am little worried the blunt hammer fix could pontentially hurt performance.  Maybe we can use MD to do a finegrain check? I draft a fix attached. Please take a look whether it is correct and free to change it. {F9879631}

The previous patch uploaded is somehow not intact. Upload it another time: F9879640: patch.txt <https://reviews.llvm.org/F9879640>

Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D66909/new/

https://reviews.llvm.org/D66909