<div dir="ltr">LGTM</div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Jan 29, 2016 at 1:03 PM, Hans Wennborg <span dir="ltr"><<a href="mailto:hans@chromium.org" target="_blank">hans@chromium.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">(actually cc'ing David this time)<br>
<br>
On Fri, Jan 29, 2016 at 1:02 PM, Hans Wennborg <<a href="mailto:hans@chromium.org">hans@chromium.org</a>> wrote:<br>
> David, OK to merge to 3.8?<br>
<br>
^^<br>
<br>
><br>
> On Fri, Jan 29, 2016 at 12:21 PM, Sanjay Patel via llvm-commits<br>
> <<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a>> wrote:<br>
>> Author: spatel<br>
>> Date: Fri Jan 29 14:21:02 2016<br>
>> New Revision: 259236<br>
>><br>
>> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=259236&view=rev" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project?rev=259236&view=rev</a><br>
>> Log:<br>
>> [InstCombine] avoid an insertelement transformation that induces the opposite extractelement fold (PR26354)<br>
>><br>
>> We would infinite loop because we created a shufflevector that was wider than<br>
>> needed and then failed to combine that with the insertelement. When subsequently<br>
>> visiting the extractelement from that shuffle, we see that it's unnecessary,<br>
>> delete it, and trigger another visit to the insertelement.<br>
</blockquote></div><br></div>