[PATCH] D106588: [CSSPGO] Tweak ICP threshold in top-down inliner

Lei Wang via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Jul 25 18:52:14 PDT 2021


wlei added a comment.

In D106588#2901845 <https://reviews.llvm.org/D106588#2901845>, @wenlei wrote:

> Thanks for working on the tuning. The switch names could be confusing. How about name them this way:
>
> - `sample-profile-icp-relative-hotness`: default to 25.
> - `sample-profile-icp-relative-hotness-skip`: default to 1, with description "Skip relative hotness check for ICP up to given number of targets." (we have `icp-csskip` for PGO ICP, which is named similarly)

Sounds good, thanks for the suggestions!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D106588



More information about the llvm-commits mailing list