]> granicus.if.org Git - llvm/commitdiff
[X86] Fix an assert that was incorrectly checking for BMI instead of AVX512VBMI.
authorCraig Topper <craig.topper@intel.com>
Sun, 26 Nov 2017 21:14:48 +0000 (21:14 +0000)
committerCraig Topper <craig.topper@intel.com>
Sun, 26 Nov 2017 21:14:48 +0000 (21:14 +0000)
The check is actually unnecessary since AVX512VBMI implies AVX512BW which is the other part of the assert.

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

lib/Target/X86/X86ISelLowering.cpp

index 892c7e24abdecb5a531c320313563dff065d0452..5f4e4d7a11370cea4d55859d32ac0a76413dc4a8 100644 (file)
@@ -2149,8 +2149,7 @@ static void Passv64i1ArgInRegs(
     const SDLoc &Dl, SelectionDAG &DAG, SDValue Chain, SDValue &Arg,
     SmallVector<std::pair<unsigned, SDValue>, 8> &RegsToPass, CCValAssign &VA,
     CCValAssign &NextVA, const X86Subtarget &Subtarget) {
-  assert((Subtarget.hasBWI() || Subtarget.hasBMI()) &&
-         "Expected AVX512BW or AVX512BMI target!");
+  assert(Subtarget.hasBWI() && "Expected AVX512BW target!");
   assert(Subtarget.is32Bit() && "Expecting 32 bit target");
   assert(Arg.getValueType() == MVT::i64 && "Expecting 64 bit value");
   assert(VA.isRegLoc() && NextVA.isRegLoc() &&