[X86] combineSetCCMOVMSK - consistently use CmpBits variable. NFCI.
authorSimon Pilgrim <llvm-dev@redking.me.uk>
Sat, 20 Jun 2020 11:35:24 +0000 (12:35 +0100)
committerSimon Pilgrim <llvm-dev@redking.me.uk>
Sat, 20 Jun 2020 11:35:24 +0000 (12:35 +0100)
The comparison value should be the same size - I've added an assert to be absolutely certain.

llvm/lib/Target/X86/X86ISelLowering.cpp

index 9ce8245..817bb42 100644 (file)
@@ -40518,6 +40518,7 @@ static SDValue combineSetCCMOVMSK(SDValue EFLAGS, X86::CondCode &CC,
 
   SDValue CmpOp = EFLAGS.getOperand(0);
   unsigned CmpBits = CmpOp.getValueSizeInBits();
+  assert(CmpBits == CmpVal.getBitWidth() && "Value size mismatch");
 
   // Peek through any truncate.
   if (CmpOp.getOpcode() == ISD::TRUNCATE)
@@ -40535,7 +40536,7 @@ static SDValue combineSetCCMOVMSK(SDValue EFLAGS, X86::CondCode &CC,
   unsigned NumEltBits = VecVT.getScalarSizeInBits();
 
   bool IsAnyOf = CmpOpcode == X86ISD::CMP && CmpVal.isNullValue();
-  bool IsAllOf = CmpOpcode == X86ISD::SUB && NumElts <= CmpVal.getBitWidth() &&
+  bool IsAllOf = CmpOpcode == X86ISD::SUB && NumElts <= CmpBits &&
                  CmpVal.isMask(NumElts);
   if (!IsAnyOf && !IsAllOf)
     return SDValue();