diff options
author | Bruno Cardoso Lopes <bruno.cardoso@gmail.com> | 2011-07-29 01:31:04 +0000 |
---|---|---|
committer | Bruno Cardoso Lopes <bruno.cardoso@gmail.com> | 2011-07-29 01:31:04 +0000 |
commit | c00f6728bc803026d406daf5efba771ecb48b5af (patch) | |
tree | a4a1244285bf517185e8abba66b3592f5ac7cc57 | |
parent | b9ba465de8a9ae43362e4857401175dc9dcc4283 (diff) | |
download | bcm5719-llvm-c00f6728bc803026d406daf5efba771ecb48b5af.tar.gz bcm5719-llvm-c00f6728bc803026d406daf5efba771ecb48b5af.zip |
Fix a bug while generating target specific VPERMIL masks: skip
undef mask elements. This fixes PR10529.
llvm-svn: 136450
-rw-r--r-- | llvm/lib/Target/X86/X86ISelLowering.cpp | 16 |
1 files changed, 12 insertions, 4 deletions
diff --git a/llvm/lib/Target/X86/X86ISelLowering.cpp b/llvm/lib/Target/X86/X86ISelLowering.cpp index f4b670be1cb..7baa9642400 100644 --- a/llvm/lib/Target/X86/X86ISelLowering.cpp +++ b/llvm/lib/Target/X86/X86ISelLowering.cpp @@ -3488,8 +3488,12 @@ static unsigned getShuffleVPERMILPSImmediate(SDNode *N) { int NumLanes = VT.getSizeInBits()/128; unsigned Mask = 0; - for (int i = 0; i < NumElts/NumLanes /* lane size */; ++i) - Mask |= SVOp->getMaskElt(i) << (i*2); + for (int i = 0; i < NumElts/NumLanes /* lane size */; ++i) { + int MaskElt = SVOp->getMaskElt(i); + if (MaskElt < 0) + continue; + Mask |= MaskElt << (i*2); + } return Mask; } @@ -3506,8 +3510,12 @@ static unsigned getShuffleVPERMILPDImmediate(SDNode *N) { unsigned Mask = 0; int LaneSize = NumElts/NumLanes; for (int l = 0; l < NumLanes; ++l) - for (int i = l*LaneSize; i < LaneSize*(l+1); ++i) - Mask |= (SVOp->getMaskElt(i)-l*LaneSize) << i; + for (int i = l*LaneSize; i < LaneSize*(l+1); ++i) { + int MaskElt = SVOp->getMaskElt(i); + if (MaskElt < 0) + continue; + Mask |= (MaskElt-l*LaneSize) << i; + } return Mask; } |