[PATCH] D102961: [CSSPGO][llvm-profgen] Report samples for untrackable frames.
Hongtao Yu via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon May 24 10:37:04 PDT 2021
hoy added inline comments.
================
Comment at: llvm/tools/llvm-profgen/PerfReader.cpp:150
+ << format("%" PRIx64, Cur->Address) << " \n";
+ T EmptyStack(Binary);
+ collectSamplesFromFrame(Cur, EmptyStack);
----------------
wlei wrote:
> Previously we always missed the context from the first frame, right? like `foo @ bar`, it will miss `foo`.
Exactly. And if `bar` is inlined into `foo`, the issue will cause samples of `bar` to be ignored. For the attached regression test, without the fix, the output profile is empty.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D102961/new/
https://reviews.llvm.org/D102961
More information about the llvm-commits
mailing list