[PATCH] D109111: [CSSPGO] Use preinliner decision by default when available

Wed Sep 1 17:38:10 PDT 2021

hoy added inline comments.

================
Comment at: llvm/lib/Transforms/IPO/SampleProfile.cpp:1314
+  if (UsePreInlinerDecision &&
+      Candidate.CalleeSamples->getContext().hasAttribute(
+          ContextShouldBeInlined))
----------------
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?

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D109111