<p dir="ltr">Hi Hans, I remember this one. I think it should be safe to pull it in, since it's self contained and fixes a real bug. </p>
<p dir="ltr">Cheers, <br>
Renato </p>
<div class="gmail_quote">On 2 Feb 2015 21:12, "Hans Wennborg" <<a href="mailto:hans@chromium.org">hans@chromium.org</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Mon, Feb 2, 2015 at 8:56 AM, Jan Wen Voung <<a href="mailto:jvoung@google.com">jvoung@google.com</a>> wrote:<br>
> Author: jvoung<br>
> Date: Mon Feb  2 10:56:50 2015<br>
> New Revision: 227809<br>
><br>
> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=227809&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=227809&view=rev</a><br>
> Log:<br>
> Fix ARM peephole optimizeCompare to avoid optimizing unsigned cmp to 0.<br>
><br>
> Summary:<br>
> Previously it only avoided optimizing signed comparisons to 0.<br>
> Sometimes the DAGCombiner will optimize the unsigned comparisons<br>
> to 0 before it gets to the peephole pass, but sometimes it doesn't.<br>
><br>
> Fix for PR22373.<br>
<br>
Renato: I think we want this for 3.6. Does it look ok to you?<br>
<br>
 - Hans<br>
</blockquote></div>