[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/

Duncan Sands
Tue, 24 Feb 2009 08:28:10 -0000

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