[llvm] [TTI][AMDGPU] Allow targets to adjust `LastCallToStaticBonus` via `getInliningLastCallToStaticBonus` (PR #111311)

Shilei Tian via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 8 13:14:46 PDT 2024


================
@@ -240,6 +240,11 @@ class GCNTTIImpl final : public BasicTTIImplBase<GCNTTIImpl> {
   bool areInlineCompatible(const Function *Caller,
                            const Function *Callee) const;
 
+  int getInliningLastCallToStaticBonus() const {
+    // InlineConstants::LastCallToStaticBonus * 11 where 11 is AMDGPU's inlining
+    // threshold multiplier.
+    return 165000;
----------------
shiltian wrote:

> Can you just take the existing multiplier and apply it to the existing threshold without the new layer?

That is what currently is doing but apparently the threshold is not large enough for a large function even after the constant bonus applied.

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


More information about the llvm-commits mailing list