[LLVMdev] [llvm-commits] [llvm] r65296 - in /llvm/trunk: include/llvm/CodeGen/ lib/CodeGen/SelectionDAG/ lib/Target/CellSPU/ lib/Target/PowerPC/ lib/Target/X86/ test/CodeGen/X86/

Scott Michel
Tue, 24 Feb 2009 17:35:45 -0000

Duncan:

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.)


-scooter

On Tue, Feb 24, 2009 at 12:28 AM, Duncan Sands <baldrick at free.fr>
wrote:

> > 3. Introduce a new ShuffleVectorSDNode that only has two SDValue
> > operands (the two input vectors), but that also contains an array of
> > ints in the node (not as operands).
> ...
> > The important part of #3 is that we really want an array of ints
> > (using -1 for undef) for the shuffle mask, not "operands".
This
> > eliminates the nastiness we have now were we need a buildvector, and
> > it eliminates the dance we have to prevent the build vector from being
> > legalized, and prevent the integer operands to it from being
legalized.
>
> This is PR2957 (which originally suggested a variadic SDNode, but it
> quickly became clear that an array of ints is better).  It would be
> great to have a volunteer for this (I don't have time).
>
> Ciao,
>
> Duncan.
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
<http://lists.llvm.org/pipermail/llvm-dev/attachments/20090224/bc2ae490/attachment.html>