[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