summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorCraig Topper <craig.topper@intel.com>2019-01-12 06:13:44 +0000
committerCraig Topper <craig.topper@intel.com>2019-01-12 06:13:44 +0000
commita69d903204377a971d73bbc096e28521e116e364 (patch)
treeeafe68d6c006f83bcf37a456f7cc711af842021c
parent0053d15d9c836576cd2f2616888b57e355aae720 (diff)
downloadbcm5719-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.cpp6
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!");
OpenPOWER on IntegriCloud