[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