diff options
| author | Jonas Paulsson <paulsson@linux.vnet.ibm.com> | 2017-03-21 10:24:14 +0000 |
|---|---|---|
| committer | Jonas Paulsson <paulsson@linux.vnet.ibm.com> | 2017-03-21 10:24:14 +0000 |
| commit | 54c7680e1fef9dac8acd55f08fb3f46e6caff400 (patch) | |
| tree | 8f0da3389c8ebfb7d15efcedca015dc7426dfd18 | |
| parent | da21170c49fb6d5bca8d04526729d821481779a4 (diff) | |
| download | bcm5719-llvm-54c7680e1fef9dac8acd55f08fb3f46e6caff400.tar.gz bcm5719-llvm-54c7680e1fef9dac8acd55f08fb3f46e6caff400.zip | |
[DAGTypeLegalizer] Handle widening truncate to vector of i1.
Previously, PromoteIntRes_TRUNCATE() did not handle the case where
the operand needs widening, which resulted in llvm_unreachable().
This patch adds the needed handling, along with a test case.
Review: Eli Friedman, Simon Pilgrim.
https://reviews.llvm.org/D31077
llvm-svn: 298357
| -rw-r--r-- | llvm/lib/CodeGen/SelectionDAG/LegalizeIntegerTypes.cpp | 22 | ||||
| -rw-r--r-- | llvm/test/CodeGen/SystemZ/vec-trunc-to-i1.ll | 37 |
2 files changed, 58 insertions, 1 deletions
diff --git a/llvm/lib/CodeGen/SelectionDAG/LegalizeIntegerTypes.cpp b/llvm/lib/CodeGen/SelectionDAG/LegalizeIntegerTypes.cpp index 39aef98362c..85068e89075 100644 --- a/llvm/lib/CodeGen/SelectionDAG/LegalizeIntegerTypes.cpp +++ b/llvm/lib/CodeGen/SelectionDAG/LegalizeIntegerTypes.cpp @@ -690,7 +690,7 @@ SDValue DAGTypeLegalizer::PromoteIntRes_TRUNCATE(SDNode *N) { case TargetLowering::TypePromoteInteger: Res = GetPromotedInteger(InOp); break; - case TargetLowering::TypeSplitVector: + case TargetLowering::TypeSplitVector: { EVT InVT = InOp.getValueType(); assert(InVT.isVector() && "Cannot split scalar types"); unsigned NumElts = InVT.getVectorNumElements(); @@ -709,6 +709,26 @@ SDValue DAGTypeLegalizer::PromoteIntRes_TRUNCATE(SDNode *N) { return DAG.getNode(ISD::CONCAT_VECTORS, dl, NVT, EOp1, EOp2); } + case TargetLowering::TypeWidenVector: { + SDValue WideInOp = GetWidenedVector(InOp); + + // Truncate widened InOp. + unsigned NumElem = WideInOp.getValueType().getVectorNumElements(); + EVT TruncVT = EVT::getVectorVT(*DAG.getContext(), + N->getValueType(0).getScalarType(), NumElem); + SDValue WideTrunc = DAG.getNode(ISD::TRUNCATE, dl, TruncVT, WideInOp); + + // Zero extend so that the elements are of same type as those of NVT + EVT ExtVT = EVT::getVectorVT(*DAG.getContext(), NVT.getVectorElementType(), + NumElem); + SDValue WideExt = DAG.getNode(ISD::ZERO_EXTEND, dl, ExtVT, WideTrunc); + + // Extract the low NVT subvector. + MVT IdxTy = TLI.getVectorIdxTy(DAG.getDataLayout()); + SDValue ZeroIdx = DAG.getConstant(0, dl, IdxTy); + return DAG.getNode(ISD::EXTRACT_SUBVECTOR, dl, NVT, WideExt, ZeroIdx); + } + } // Truncate to NVT instead of VT return DAG.getNode(ISD::TRUNCATE, dl, NVT, Res); diff --git a/llvm/test/CodeGen/SystemZ/vec-trunc-to-i1.ll b/llvm/test/CodeGen/SystemZ/vec-trunc-to-i1.ll new file mode 100644 index 00000000000..705fe3dbac9 --- /dev/null +++ b/llvm/test/CodeGen/SystemZ/vec-trunc-to-i1.ll @@ -0,0 +1,37 @@ +; NOTE: Assertions have been autogenerated by utils/update_llc_test_checks.py +; +; RUN: llc < %s -mtriple=s390x-linux-gnu -mcpu=z13 | FileCheck %s +; +; Check that a widening truncate to a vector of i1 elements can be handled. + + +define void @pr32275(<4 x i8> %B15) { +; CHECK-LABEL: pr32275: +; CHECK: # BB#0: # %BB +; CHECK-NEXT: vrepif %v0, 1 +; CHECK-NEXT: .LBB0_1: # %CF34 +; CHECK-NEXT: # =>This Inner Loop Header: Depth=1 +; CHECK-NEXT: vlgvb %r0, %v24, 3 +; CHECK-NEXT: vlgvb %r1, %v24, 1 +; CHECK-NEXT: vlvgp %v1, %r1, %r0 +; CHECK-NEXT: vlgvb %r0, %v24, 0 +; CHECK-NEXT: vlvgf %v1, %r0, 0 +; CHECK-NEXT: vlgvb %r0, %v24, 2 +; CHECK-NEXT: vlvgf %v1, %r0, 2 +; CHECK-NEXT: vn %v1, %v1, %v0 +; CHECK-NEXT: vlgvf %r0, %v1, 3 +; CHECK-NEXT: tmll %r0, 1 +; CHECK-NEXT: jne .LBB0_1 +; CHECK-NEXT: # BB#2: # %CF36 +; CHECK-NEXT: br %r14 +BB: + br label %CF34 + +CF34: + %Tr24 = trunc <4 x i8> %B15 to <4 x i1> + %E28 = extractelement <4 x i1> %Tr24, i32 3 + br i1 %E28, label %CF34, label %CF36 + +CF36: + ret void +} |

