[PATCH] D153797: [CSSPGO][Preinliner] Bump up the threshold to favor previous compiler inline decision.

Hongtao Yu via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 26 12:03:16 PDT 2023


hoy added inline comments.


================
Comment at: llvm/tools/llvm-profgen/CSPreInliner.cpp:193
+    if (WasInlined)
+      SampleThreshold *= 100;
   }
----------------
wenlei wrote:
> Make this a tunable value via a switch? 
Sounds good. 

Forgot to mention that smaller numbers like 10, 25, 50 don't give as good win as 100 gives.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D153797/new/

https://reviews.llvm.org/D153797



More information about the llvm-commits mailing list