[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 14:04:06 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:

That is gonna change all the targets, which I'm not sure if that's a good idea. That is also the motivation to have a target dependent API instead of assuming all targets want to inline the last call.

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


More information about the llvm-commits mailing list