Duncan:<br><br>I'm still stymied how this whole thread ended up about shuffle vector nodes, when the original problem was my build vector patch. I'm still working on backing the build vector patch out (it isn't clean with all of the intervening commits and I have pressing management tasks which command my attention.)<br>
<br><br>-scooter<br><br><div class="gmail_quote">On Tue, Feb 24, 2009 at 12:28 AM, Duncan Sands <span dir="ltr"><<a href="mailto:baldrick@free.fr">baldrick@free.fr</a>></span> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div class="Ih2E3d">> 3. Introduce a new ShuffleVectorSDNode that only has two SDValue<br>
> operands (the two input vectors), but that also contains an array of<br>
> ints in the node (not as operands).<br>
</div>...<br>
<div class="Ih2E3d">> The important part of #3 is that we really want an array of ints<br>
> (using -1 for undef) for the shuffle mask, not "operands".  This<br>
> eliminates the nastiness we have now were we need a buildvector, and<br>
> it eliminates the dance we have to prevent the build vector from being<br>
> legalized, and prevent the integer operands to it from being legalized.<br>
<br>
</div>This is PR2957 (which originally suggested a variadic SDNode, but it<br>
quickly became clear that an array of ints is better).  It would be<br>
great to have a volunteer for this (I don't have time).<br>
<br>
Ciao,<br>
<font color="#888888"><br>
Duncan.<br>
</font></blockquote></div><br>