Hi,<br><br>LGTM too. It makes sense to avoid churn in the lit checks unnecessarily. <br><br>Cheers,<br><br>James<br><div class="gmail_quote"><div dir="ltr">On Sat, 19 Dec 2015 at 09:29, MinSeong KIM via llvm-commits <<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">MinSeongKIM accepted this revision.<br>
MinSeongKIM added a comment.<br>
This revision is now accepted and ready to land.<br>
<br>
Hi Chad, LGTM.<br>
This makes sense to me as the tests broken assume no post RA scheduler.<br>
Also I did some work to mute the errors from the broken tests (Please see <a href="http://reviews.llvm.org/D15666" rel="noreferrer" target="_blank">http://reviews.llvm.org/D15666</a> the revision is not for upstreaming, but to help the reviewers of "this revision"). I am not sure the changes are all legal and safe, but instruction orders for most of them do not seem matter. I hope it helps.<br>
<br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="http://reviews.llvm.org/D15652" rel="noreferrer" target="_blank">http://reviews.llvm.org/D15652</a><br>
<br>
<br>
<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
</blockquote></div>