[PATCH] D109111: [CSSPGO] Use preinliner decision by default when available
Hongtao Yu via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Sep 1 17:43:56 PDT 2021
hoy accepted this revision.
hoy added inline comments.
This revision is now accepted and ready to land.
================
Comment at: llvm/lib/Transforms/IPO/SampleProfile.cpp:1314
+ if (UsePreInlinerDecision &&
+ Candidate.CalleeSamples->getContext().hasAttribute(
+ ContextShouldBeInlined))
----------------
wenlei wrote:
> hoy wrote:
> > wenlei wrote:
> > > hoy wrote:
> > > > Will non-inlined context be merged into the base profile by the preinliner? Wondering if we can have contexts without `ContextShouldBeInlined` in the profile after processed by the preinliner.
> > > Not inlined context will all be merged currently. See the comment I added above.
> > So we'll not see the context profile here? Can such context only be seen when the preinliner is off?
> Correct. When preinliner is on in llvm-profgen, we will only see context profile if preinliner decides to inline. When preinliner is off, we will see all context profiles.
Thanks for the confirmation. Can you update the comment to reflect that?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D109111/new/
https://reviews.llvm.org/D109111
More information about the llvm-commits
mailing list