This change was very specific to the loop that the code is using. While its in shuffle code its not looking at the shuffle indices. It's just checking to see if the 128-bit lanes are linked using a variable that's specific to this loop. The value is either -1(not linked) or a positive value(linked).<br>
<br><div class="gmail_quote">On Mon, Jan 21, 2013 at 12:22 PM,  <span dir="ltr"><<a href="mailto:dag@cray.com" target="_blank">dag@cray.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Craig Topper <<a href="mailto:craig.topper@gmail.com">craig.topper@gmail.com</a>> writes:<br>
<br>
> Author: ctopper<br>
> Date: Mon Jan 21 01:25:16 2013<br>
> New Revision: 173010<br>
><br>
> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=173010&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=173010&view=rev</a><br>
> Log:<br>
> Use <0 checks in place of ==-1 because it results in simpler code.<br>
<br>
Is that perhaps obscuring the intent?  -1 is a special UNDEF value.  I<br>
preferred the old code where we had an isUNDEF helper.<br>
<span class="HOEnZb"><font color="#888888"><br>
                                    -David<br>
</font></span></blockquote></div><br><br clear="all"><div><br></div>-- <br>~Craig