<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Oct 6, 2015 at 1:56 PM, Diego Novillo <span dir="ltr"><<a href="mailto:dnovillo@google.com" target="_blank" class="cremed">dnovillo@google.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="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote"><span class="">On Tue, Oct 6, 2015 at 1:52 PM, Sanjoy Das <span dir="ltr"><<a href="mailto:sanjoy@playingwithpointers.com" target="_blank" class="cremed">sanjoy@playingwithpointers.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><span><br></span>
If you're going by "<jokereph_> dnovillo: I just commited r249431 based on sanjoyd commit, you’ll have to revert this one first." on IRC, I think what he meant was that 249431 builds on the br-phi change, not that it fixes the ICE. :)<br></blockquote><div><br></div></span><div>Oh, drat.  Yeah, I misread.  I'm still having this issue.  I was at an earlier point than r249231, so I could just revert your patch.  I will try <a href="http://reviews.llvm.org/D13460" target="_blank" class="cremed">http://reviews.llvm.org/D13460</a>.</div></div></div></div>
</blockquote></div><br></div><div class="gmail_extra">This new patch fixes the problem.  Any idea when you'll be able to land it?  While I can build my own compiler with this, I can't apply out-of-trunk patches to our internal build.  I'd need to revert 249211 and 249431 unless you think D13460 will be committed soon.</div><div class="gmail_extra"><br></div><div class="gmail_extra"><br></div><div class="gmail_extra">Thanks.  Diego.</div></div>