[llvm] [FuncSpec] Only compute Latency bonus when necessary (PR #113159)

[Alexandros Lamprineas via llvm-commits]

================
@@ -889,18 +910,20 @@ bool FunctionSpecializer::findSpecializations(Function *F, unsigned FuncSize,
       unsigned Score = 0;
       InstCostVisitor Visitor = getInstCostVisitorFor(F);
       for (ArgInfo &A : S.Args) {
-        CodeSize += Visitor.getCodeSizeBonus(A.Formal, A.Actual);
+        CodeSize += Visitor.getCodeSizeSavingsForArg(A.Formal, A.Actual);
         Score += getInliningBonus(A.Formal, A.Actual);
       }
-      CodeSize += Visitor.getCodeSizeBonusFromPendingPHIs();
+      CodeSize += Visitor.getCodeSizeSavingsFromPendingPHIs();
 
       LLVM_DEBUG(dbgs() << "FnSpecialization: Specialization bonus {CodeSize = "
----------------
labrinea wrote:

Can we sink this inside IsProfitable, so that all the debug messages about profitability are in the same place? Perhaps we can split it in three messages: one for Inlining, one for CodeSize and one for Latency.

https://github.com/llvm/llvm-project/pull/113159