Incorporate review comments

justinfargnoli · justinfargnoli · commit b29101cd8204 · 2025-08-13T00:39:11.000Z
diff --git a/llvm/lib/Analysis/InlineCost.cpp b/llvm/lib/Analysis/InlineCost.cpp
@@ -3277,7 +3277,8 @@ InlineCost llvm::getInlineCost(
   }
 
   if (InlineAllViableCalls && isInlineViable(*Callee).isSuccess())
-    return llvm::InlineCost::getAlways("inline all viable calls");
+    return llvm::InlineCost::getAlways(
+        "Inlining forced by --inline-all-viable-calls");
 
   LLVM_DEBUG(llvm::dbgs() << "      Analyzing call of " << Callee->getName()
                           << "... (caller:" << Call.getCaller()->getName()
diff --git a/llvm/test/Transforms/Inline/inline-all-viable-calls.ll b/llvm/test/Transforms/Inline/inline-all-viable-calls.ll
@@ -1,4 +1,3 @@
-; NOTE: Assertions have been autogenerated by utils/update_test_checks.py UTC_ARGS: --version 5
 ; RUN: opt -passes=inline -inline-threshold=0 -inline-all-viable-calls -S < %s | FileCheck %s
 
 ; Check that viable calls that are beyond the cost threshold are still inlined.

Original file line number	Diff line number	Diff line change
`@@ -3277,7 +3277,8 @@ InlineCost llvm::getInlineCost(`
`3277`	`3277`	`}`
`3278`	`3278`
`3279`	`3279`	`if (InlineAllViableCalls && isInlineViable(*Callee).isSuccess())`
`3280`		`- return llvm::InlineCost::getAlways("inline all viable calls");`
	`3280`	`+ return llvm::InlineCost::getAlways(`
	`3281`	`+ "Inlining forced by --inline-all-viable-calls");`
`3281`	`3282`
`3282`	`3283`	`LLVM_DEBUG(llvm::dbgs() << " Analyzing call of " << Callee->getName()`
`3283`	`3284`	`<< "... (caller:" << Call.getCaller()->getName()`
Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,3 @@`
`1`		`-; NOTE: Assertions have been autogenerated by utils/update_test_checks.py UTC_ARGS: --version 5`
`2`	`1`	`; RUN: opt -passes=inline -inline-threshold=0 -inline-all-viable-calls -S < %s \| FileCheck %s`
`3`	`2`
`4`	`3`	`; Check that viable calls that are beyond the cost threshold are still inlined.`