diff options
| author | Simon Pilgrim <llvm-dev@redking.me.uk> | 2017-03-05 15:52:18 +0000 |
|---|---|---|
| committer | Simon Pilgrim <llvm-dev@redking.me.uk> | 2017-03-05 15:52:18 +0000 |
| commit | 584d6d9d9109ff90aa34d064903ee0d572a7fc5a (patch) | |
| tree | 62532705cda505baeba9bc16eeeb9643603baca5 | |
| parent | 495ad0b2a0f091db203747a87bbf5a34fd3f84da (diff) | |
| download | bcm5719-llvm-584d6d9d9109ff90aa34d064903ee0d572a7fc5a.tar.gz bcm5719-llvm-584d6d9d9109ff90aa34d064903ee0d572a7fc5a.zip | |
[SelectionDAG] Fix vector splitting for *_EXTEND_VECTOR_INREG instructions
Found by fuzz testing after rL296985 landed
llvm-svn: 296989
| -rw-r--r-- | llvm/lib/CodeGen/SelectionDAG/LegalizeVectorTypes.cpp | 7 | ||||
| -rw-r--r-- | llvm/test/CodeGen/X86/split-extend-vector-inreg.ll | 47 |
2 files changed, 53 insertions, 1 deletions
diff --git a/llvm/lib/CodeGen/SelectionDAG/LegalizeVectorTypes.cpp b/llvm/lib/CodeGen/SelectionDAG/LegalizeVectorTypes.cpp index 0ddae3648d0..358d4264803 100644 --- a/llvm/lib/CodeGen/SelectionDAG/LegalizeVectorTypes.cpp +++ b/llvm/lib/CodeGen/SelectionDAG/LegalizeVectorTypes.cpp @@ -963,7 +963,12 @@ void DAGTypeLegalizer::SplitVecRes_ExtVecInRegOp(SDNode *N, SDValue &Lo, SDLoc dl(N); SDValue InLo, InHi; - GetSplitVector(N0, InLo, InHi); + + if (getTypeAction(N0.getValueType()) == TargetLowering::TypeSplitVector) + GetSplitVector(N0, InLo, InHi); + else + std::tie(InLo, InHi) = DAG.SplitVectorOperand(N, 0); + EVT InLoVT = InLo.getValueType(); unsigned InNumElements = InLoVT.getVectorNumElements(); diff --git a/llvm/test/CodeGen/X86/split-extend-vector-inreg.ll b/llvm/test/CodeGen/X86/split-extend-vector-inreg.ll new file mode 100644 index 00000000000..692cbdb00be --- /dev/null +++ b/llvm/test/CodeGen/X86/split-extend-vector-inreg.ll @@ -0,0 +1,47 @@ +; NOTE: Assertions have been autogenerated by utils/update_llc_test_checks.py +; RUN: llc < %s -mtriple=i686-unknown -mattr=+avx | FileCheck %s --check-prefix=X32 +; RUN: llc < %s -mtriple=x86_64-unknown -mattr=+avx | FileCheck %s --check-prefix=X64 + +define <4 x i64> @autogen_SD88863() { +; X32-LABEL: autogen_SD88863: +; X32: # BB#0: # %BB +; X32-NEXT: vpshufd {{.*#+}} xmm0 = xmm0[2,3,0,1] +; X32-NEXT: vinsertf128 $1, %xmm0, %ymm0, %ymm0 +; X32-NEXT: vxorpd %ymm1, %ymm1, %ymm1 +; X32-NEXT: vblendpd {{.*#+}} ymm0 = ymm1[0,1],ymm0[2],ymm1[3] +; X32-NEXT: movb $1, %al +; X32-NEXT: .p2align 4, 0x90 +; X32-NEXT: .LBB0_1: # %CF +; X32-NEXT: # =>This Inner Loop Header: Depth=1 +; X32-NEXT: testb %al, %al +; X32-NEXT: jne .LBB0_1 +; X32-NEXT: # BB#2: # %CF240 +; X32-NEXT: retl +; +; X64-LABEL: autogen_SD88863: +; X64: # BB#0: # %BB +; X64-NEXT: vpshufd {{.*#+}} xmm0 = xmm0[2,3,0,1] +; X64-NEXT: vinsertf128 $1, %xmm0, %ymm0, %ymm0 +; X64-NEXT: vxorpd %ymm1, %ymm1, %ymm1 +; X64-NEXT: vblendpd {{.*#+}} ymm0 = ymm1[0,1],ymm0[2],ymm1[3] +; X64-NEXT: movb $1, %al +; X64-NEXT: .p2align 4, 0x90 +; X64-NEXT: .LBB0_1: # %CF +; X64-NEXT: # =>This Inner Loop Header: Depth=1 +; X64-NEXT: testb %al, %al +; X64-NEXT: jne .LBB0_1 +; X64-NEXT: # BB#2: # %CF240 +; X64-NEXT: retq +BB: + %I26 = insertelement <4 x i64> undef, i64 undef, i32 2 + br label %CF + +CF: + %E66 = extractelement <4 x i64> %I26, i32 1 + %I68 = insertelement <4 x i64> zeroinitializer, i64 %E66, i32 2 + %Cmp72 = icmp eq i32 0, 0 + br i1 %Cmp72, label %CF, label %CF240 + +CF240: + ret <4 x i64> %I68 +} |

