[PATCH] D96197: [CSSPGO] Add switches to control prelink/postlink inline separately

Wei Mi via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 9 23:01:51 PST 2021


wmi added a comment.

Have you tried tuning the inline param instead of disabling CGSCC inlining in prelto? From our experience, thinlto wants some inlining happening in prelink phase especially for smaller callees so that the heuristic of choosing functions to import can be more effective. Current sampleFDO also turned off hot callsite inline heuristic in CGSCC inlining by setting the inline param, in order to prevent optimizations in prelink from obstructing profile annotation in postlink, but keep the other params the same. Feel using inline param for the tuning will be more flexible.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D96197



More information about the llvm-commits mailing list