<div dir="ltr"><div>r233351, r233353 and r233355 merged to release_36 as r234303.<br></div>The last two commits merged move a test from CodeGen/Generic to CodeGen/X86.<br></div><br><div class="gmail_quote">On Fri, Apr 3, 2015 at 2:56 PM Paweł Bylica <<a href="mailto:chfast@gmail.com">chfast@gmail.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_quote">On Fri, Mar 27, 2015 at 9:47 PM Tom Stellard <<a href="mailto:tom@stellard.net" target="_blank">tom@stellard.net</a>> wrote:<br></div></div><div dir="ltr"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">LGTM, go ahead and merge this into the 3.6 branch.<br></blockquote></div></div><div dir="ltr"><div class="gmail_quote"><div>Can anyone merge it for me? Don't have write access to repo.</div></div></div><div dir="ltr"><div class="gmail_quote"><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
On Fri, Mar 27, 2015 at 12:21:47PM -0700, Andrew Trick wrote:<br>
> Fine with me.<br>
> Andy<br>
><br>
> > On Mar 27, 2015, at 6:16 AM, Paweł Bylica <<a href="mailto:chfast@gmail.com" target="_blank">chfast@gmail.com</a>> wrote:<br>
> ><br>
> > Hi Andrew, Quentin, Owen & Tom,<br>
> ><br>
> > I'm not sure who is the owner of the code but I worked with Quentin and Andrew on the solution. I would like the fix to be included in 3.6.1.<br>
> ><br>
> > - Paweł<br>
> ><br>
> > r233351:<br>
> ><br>
> > Bug: <a href="https://llvm.org/bugs/show_bug.cgi?id=22304" target="_blank">https://llvm.org/bugs/show_<u></u>bug<u></u>.cgi?id=22304</a> <<a href="https://llvm.org/bugs/show_bug.cgi?id=22304" target="_blank">https://llvm.org/bugs/show_<u></u>bu<u></u>g.cgi?id=22304</a>><br>
> > Review: <a href="http://reviews.llvm.org/D8556" target="_blank">http://reviews.llvm.org/D8556</a> <<a href="http://reviews.llvm.org/D8556" target="_blank">http://reviews.llvm.org/D8556</a><u></u><u></u>><br>
> ><br>
> > Author: atrick<br>
> > Date: Thu Mar 26 22:44:13 2015<br>
> > New Revision: 233351<br>
> ><br>
> > URL: <a href="http://llvm.org/viewvc/llvm-project?rev=233351&view=rev" target="_blank">http://llvm.org/viewvc/llvm-<u></u>pr<u></u>oject?rev=233351&view=rev</a> <<a href="http://llvm.org/viewvc/llvm-project?rev=233351&view=rev" target="_blank">http://llvm.org/viewvc/llvm-<u></u>p<u></u>roject?rev=233351&view=rev</a>><br>
> > Log:<br>
> > Fix a bug in SelectionDAG scheduling backtracking code: PR22304.<br>
> ><br>
> > It can happen (by line CurSU->isPending = true; // This SU is not in<br>
> > AvailableQueue right now.) that a SUnit is mark as available but is<br>
> > not in the AvailableQueue. For SUnit being selected for scheduling<br>
> > both conditions must be met.<br>
> ><br>
> > This patch mainly defensively protects from invalid removing a node<br>
> > from a queue. Sometimes nodes are marked isAvailable but are not in<br>
> > the queue because they have been defered due to some hazard.<br>
> ><br>
> > Patch by Pawel Bylica!<br>
> ><br>
> > Added:<br>
> >     llvm/trunk/test/CodeGen/<u></u>Gener<u></u>ic/scheduler-<u></u>backtracking.ll<br>
> > Modified:<br>
> >     llvm/trunk/lib/CodeGen/<u></u>Select<u></u>ionDAG/<u></u>ScheduleDAGRRList.cpp<br>
><br>
</blockquote></div></div></blockquote></div>