davidxl added a comment.
Can you add a test case where there is more than one level of inline contexts
for the callsite?
================
Comment at: llvm/lib/Analysis/InlineAdvisor.cpp:391
+ Remark << ore::NV("Caller", &Caller);
+ if (ProfileGuidedInline)
+ Remark << " by profile guided inliner";
----------------
is this necessary? User should know if their build has profile or not.
What is more useful is when PGO is on, but some callsite does not have profile
data, then it is worth reporting.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D82213/new/
https://reviews.llvm.org/D82213
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
