[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:30:31 PDT 2021
hoy added inline comments.
================
Comment at: llvm/lib/Transforms/IPO/SampleProfile.cpp:1314
+ if (UsePreInlinerDecision &&
+ Candidate.CalleeSamples->getContext().hasAttribute(
+ ContextShouldBeInlined))
----------------
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.
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