]> granicus.if.org Git - llvm/commitdiff
[X86] Remove invalid code from LowerVSELECT.
authorCraig Topper <craig.topper@intel.com>
Sat, 28 Oct 2017 23:10:13 +0000 (23:10 +0000)
committerCraig Topper <craig.topper@intel.com>
Sat, 28 Oct 2017 23:10:13 +0000 (23:10 +0000)
This code attempted to say that v8i16/v16i16 VSELECT is legal if BWI and VLX are enabled, but the only way we could reach this point is if the condition was not a vXi1 type. Which means it really wasn't legal.

We don't have any tests that exercise this code. So I'm hoping it wasn't really reachable.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@316851 91177308-0d34-0410-b5e6-96231b3b80d8

lib/Target/X86/X86ISelLowering.cpp

index 0b3615843a31b073fe2511f2929deed99be5da57..900c99be8088642be74c7c842cd70408a3eee535 100644 (file)
@@ -14232,10 +14232,6 @@ SDValue X86TargetLowering::LowerVSELECT(SDValue Op, SelectionDAG &DAG) const {
 
   case MVT::v8i16:
   case MVT::v16i16:
-    // AVX-512 BWI and VLX features support VSELECT with i16 elements.
-    if (Subtarget.hasBWI() && Subtarget.hasVLX())
-      return Op;
-
     // FIXME: We should custom lower this by fixing the condition and using i8
     // blends.
     return SDValue();