From 3b1d40d0cd99a056efc53e3728287a6ad39950ec Mon Sep 17 00:00:00 2001 From: Craig Topper Date: Sat, 12 Jan 2019 06:13:44 +0000 Subject: [PATCH] [X86] Remove unnecessary code from getMaskNode. We no longer need to extend mask scalars before bitcasting them to vXi1. This was only needed for the truncate intrinsics. And was really a bug in our lowering of them. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@350991 91177308-0d34-0410-b5e6-96231b3b80d8 --- lib/Target/X86/X86ISelLowering.cpp | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/lib/Target/X86/X86ISelLowering.cpp b/lib/Target/X86/X86ISelLowering.cpp index 5766773f129..b9c562008a4 100644 --- a/lib/Target/X86/X86ISelLowering.cpp +++ b/lib/Target/X86/X86ISelLowering.cpp @@ -21332,11 +21332,7 @@ static SDValue getMaskNode(SDValue Mask, MVT MaskVT, if (X86::isZeroNode(Mask)) return DAG.getConstant(0, dl, MaskVT); - if (MaskVT.bitsGT(Mask.getSimpleValueType())) { - // Mask should be extended - Mask = DAG.getNode(ISD::ANY_EXTEND, dl, - MVT::getIntegerVT(MaskVT.getSizeInBits()), Mask); - } + assert(MaskVT.bitsLE(Mask.getSimpleValueType()) && "Unexpected mask size!"); if (Mask.getSimpleValueType() == MVT::i64 && Subtarget.is32Bit()) { assert(MaskVT == MVT::v64i1 && "Expected v64i1 mask!"); -- 2.50.1