diff options
author | Craig Topper <craig.topper@intel.com> | 2019-01-12 06:13:44 +0000 |
---|---|---|
committer | Craig Topper <craig.topper@intel.com> | 2019-01-12 06:13:44 +0000 |
commit | a69d903204377a971d73bbc096e28521e116e364 (patch) | |
tree | eafe68d6c006f83bcf37a456f7cc711af842021c | |
parent | 0053d15d9c836576cd2f2616888b57e355aae720 (diff) | |
download | bcm5719-llvm-a69d903204377a971d73bbc096e28521e116e364.tar.gz bcm5719-llvm-a69d903204377a971d73bbc096e28521e116e364.zip |
[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.
llvm-svn: 350991
-rw-r--r-- | llvm/lib/Target/X86/X86ISelLowering.cpp | 6 |
1 files changed, 1 insertions, 5 deletions
diff --git a/llvm/lib/Target/X86/X86ISelLowering.cpp b/llvm/lib/Target/X86/X86ISelLowering.cpp index 5766773f129..b9c562008a4 100644 --- a/llvm/lib/Target/X86/X86ISelLowering.cpp +++ b/llvm/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!"); |