Index: llvm/trunk/include/llvm/Analysis/InlineCost.h =================================================================== --- llvm/trunk/include/llvm/Analysis/InlineCost.h +++ llvm/trunk/include/llvm/Analysis/InlineCost.h @@ -124,7 +124,7 @@ int DefaultThreshold; /// Threshold to use for callees with inline hint. - int HintThreshold; + Optional HintThreshold; /// Threshold to use for cold callees. Optional ColdThreshold; @@ -136,7 +136,7 @@ Optional OptMinSizeThreshold; /// Threshold to use when the callsite is considered hot. - int HotCallSiteThreshold; + Optional HotCallSiteThreshold; }; /// Generate the parameters to tune the inline cost analysis based only on the Index: llvm/trunk/lib/Analysis/InlineCost.cpp =================================================================== --- llvm/trunk/lib/Analysis/InlineCost.cpp +++ llvm/trunk/lib/Analysis/InlineCost.cpp @@ -624,6 +624,11 @@ return B ? std::min(A, B.getValue()) : A; }; + // return max(A, B) if B is valid. + auto MaxIfValid = [](int A, Optional B) { + return B ? std::max(A, B.getValue()) : A; + }; + // Use the OptMinSizeThreshold or OptSizeThreshold knob if they are available // and reduce the threshold if the caller has the necessary attribute. if (Caller->optForMinSize()) @@ -644,11 +649,10 @@ bool InlineHint = Callee.hasFnAttribute(Attribute::InlineHint) || PSI->isHotFunction(&Callee); if (InlineHint && !Caller->optForMinSize()) - Threshold = std::max(Threshold, Params.HintThreshold); + Threshold = MaxIfValid(Threshold, Params.HintThreshold); - if (HotCallsite && HotCallSiteThreshold > Threshold && - !Caller->optForMinSize()) - Threshold = std::max(Threshold, Params.HotCallSiteThreshold); + if (HotCallsite && !Caller->optForMinSize()) + Threshold = MaxIfValid(Threshold, Params.HotCallSiteThreshold); bool ColdCallee = PSI->isColdFunction(&Callee); // For cold callees, use the ColdThreshold knob if it is available and reduces