<div dir="ltr"><div>Friendly ping. Thanks again for helping with this patch!<br></div><div><br></div><div>Dan</div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Oct 17, 2017 at 1:28 PM, Dan Gohman <span dir="ltr"><<a href="mailto:sunfish@mozilla.com" target="_blank">sunfish@mozilla.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="auto"><div>I haven't rebased the patch; I tested it on an older tree.</div><div dir="auto"><br></div><div dir="auto">Thanks for your help!</div><span class="HOEnZb"><font color="#888888"><div dir="auto"><br></div></font></span><div dir="auto"><span class="HOEnZb"><font color="#888888">Dan</font></span><div><div class="h5"><br><div class="gmail_extra" dir="auto"><br><div class="gmail_quote">On Oct 17, 2017 1:02 PM, "Daniel Berlin" <<a href="mailto:dberlin@dberlin.org" target="_blank">dberlin@dberlin.org</a>> wrote:<br type="attribution"><blockquote class="m_-2367864021416973442quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hey, sorry, missed your email.<div>Did you rebase it already?<br>If so, can you just post your rebase, and i'll shepherd it through review/test/fix it?</div><div><br></div><div>If not, i'll rebase it.</div><div><br></div></div><div class="m_-2367864021416973442elided-text"><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Oct 17, 2017 at 10:27 AM, Dan Gohman 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">sunfish added a comment.<br>
<br>
Friendly ping :-). I confirmed that the MemorySSA-based patch does implement the optimization. In fact, it tries //not// to, however it doesn't try hard enough, and the optimization happens anyway. It passes all the tests here as-is.<br>
<div class="m_-2367864021416973442m_-8590963759378647498HOEnZb"><div class="m_-2367864021416973442m_-8590963759378647498h5"><br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="https://reviews.llvm.org/D38374" rel="noreferrer" target="_blank">https://reviews.llvm.org/D3837<wbr>4</a><br>
<br>
<br>
<br>
</div></div></blockquote></div><br></div>
</div></blockquote></div><br></div></div></div></div></div>
</blockquote></div><br></div>