<div dir="ltr">It seems like this patch should be reverted. Recently we have no reliable way to determine the PHI (or whatever else) divergence on the MI level.<div>The only correct way is to add the DA algorithm that re-computes all the MI divergence. </div><div>Since we're going to remove the SGPRFix stuff as soon as diverence driven ISel is ready I consider this patch is not necessary.</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Sat, Mar 17, 2018 at 6:20 PM, Alexander via Phabricator <span dir="ltr"><<a href="mailto:reviews@reviews.llvm.org" target="_blank">reviews@reviews.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">alex-t added a comment.<br>
<div class="HOEnZb"><div class="h5"><br>
I m looking on this.<br>
<br>
17 марта 2018 г. 3:27 PM пользователь "Nicolai Hähnle via Phabricator" <<br>
<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> написал:<br>
<br>
> nhaehnle added a comment.<br>
><br>
> Is this related to <a href="https://reviews.llvm.org/D40547" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D40547</a>? I sent a ping on<br>
>  that, but no response...<br>
><br>
> Repository:<br>
><br>
>   rL LLVM<br>
><br>
> <a href="https://reviews.llvm.org/D40556" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D40556</a><br>
<br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="https://reviews.llvm.org/D40556" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D40556</a><br>
<br>
<br>
<br>
</div></div></blockquote></div><br></div>