commit 3b1d40d0cd99a056efc53e3728287a6ad39950ec
author Craig Topper <craig.topper@intel.com> Sat Jan 12 06:13:44 2019 +0000
committer Craig Topper <craig.topper@intel.com> Sat Jan 12 06:13:44 2019 +0000
tree 772eaf5821fb5bb4567b9935b962cf2392da562f
parent 4967aab1dc2d220ffff87ddc08923cc0e27c8567

[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

diff --git a/lib/Target/X86/X86ISelLowering.cpp b/lib/Target/X86/X86ISelLowering.cpp
index 5766773..b9c5620 100644
--- a/lib/Target/X86/X86ISelLowering.cpp
+++ b/lib/Target/X86/X86ISelLowering.cpp
@@ -21332,11 +21332,7 @@
   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!");