[CostModel][TTI] Replace BAD_ICMP_PREDICATE with ICMP_NE for generic …

…smulo/umulo cost expansion Match the predicate used in TargetLowering::expandMULO to detect overflow
llvm · Oct 6, 2021 · 2ced9a4 · 2ced9a4
1 parent 7bd097f
commit 2ced9a4
Show file tree

Hide file tree

Showing 3 changed files with 195 additions and 196 deletions.
diff --git a/llvm/include/llvm/CodeGen/BasicTTIImpl.h b/llvm/include/llvm/CodeGen/BasicTTIImpl.h
@@ -1846,9 +1846,8 @@ class BasicTTIImplBase : public TargetTransformInfoImplCRTPBase<T> {
                                                 CostKind, TTI::OK_AnyValue,
                                                 TTI::OK_UniformConstantValue);
 
-      Cost +=
-          thisT()->getCmpSelInstrCost(BinaryOperator::ICmp, MulTy, OverflowTy,
-                                      CmpInst::BAD_ICMP_PREDICATE, CostKind);
+      Cost += thisT()->getCmpSelInstrCost(
+          BinaryOperator::ICmp, MulTy, OverflowTy, CmpInst::ICMP_NE, CostKind);
       return Cost;
     }
     case Intrinsic::ctpop: