Avoid crashing on a variable-index insertelement with element type i16.
authorEli Friedman <eli.friedman@gmail.com>
Sat, 6 Jun 2009 06:32:50 +0000 (06:32 +0000)
committerEli Friedman <eli.friedman@gmail.com>
Sat, 6 Jun 2009 06:32:50 +0000 (06:32 +0000)
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@72991 91177308-0d34-0410-b5e6-96231b3b80d8

lib/Target/X86/X86ISelLowering.cpp
test/CodeGen/X86/2009-06-05-VariableIndexInsert.ll [new file with mode: 0644]

index ec6a5bad5b7428d6308b667869f73f10c2c8b879..ef60ff5e09272d0a6862576e4abc2acbb7506b6f 100644 (file)
@@ -4272,7 +4272,7 @@ X86TargetLowering::LowerINSERT_VECTOR_ELT(SDValue Op, SelectionDAG &DAG) {
   SDValue N1 = Op.getOperand(1);
   SDValue N2 = Op.getOperand(2);
 
-  if (EVT.getSizeInBits() == 16) {
+  if (EVT.getSizeInBits() == 16 && isa<ConstantSDNode>(N2)) {
     // Transform it so it match pinsrw which expects a 16-bit value in a GR32
     // as its second argument.
     if (N1.getValueType() != MVT::i32)
diff --git a/test/CodeGen/X86/2009-06-05-VariableIndexInsert.ll b/test/CodeGen/X86/2009-06-05-VariableIndexInsert.ll
new file mode 100644 (file)
index 0000000..2e3f195
--- /dev/null
@@ -0,0 +1,11 @@
+; RUN: llvm-as < %s | llc
+
+define <2 x i64> @_mm_insert_epi16(<2 x i64> %a, i32 %b, i32 %imm) nounwind readnone {
+entry:
+       %conv = bitcast <2 x i64> %a to <8 x i16>               ; <<8 x i16>> [#uses=1]
+       %conv2 = trunc i32 %b to i16            ; <i16> [#uses=1]
+       %and = and i32 %imm, 7          ; <i32> [#uses=1]
+       %vecins = insertelement <8 x i16> %conv, i16 %conv2, i32 %and           ; <<8 x i16>> [#uses=1]
+       %conv6 = bitcast <8 x i16> %vecins to <2 x i64>         ; <<2 x i64>> [#uses=1]
+       ret <2 x i64> %conv6
+}