[Lilit summer student development] Flamegraph integration - #192
[Lilit summer student development] Flamegraph integration#192oshadura wants to merge 18 commits into
Conversation
|
Rebased and readapted from #94 |
|
@oshadura I bet it's a "current working directory" issue. You can check the verbose |
|
Looks like the behavior of |
Co-Authored-By: Vassil Vassilev <v.g.vassilev@gmail.com>
It could be invocated from Cmake or as a standalone.
2194071 to
63aca79
Compare
… cpu + better dependencies management
|
Thank you @oshadura and Lilit for this work, and sorry it sat unresolved for so long! Profiling the benchmarks is still something we very much want to encourage, but in the meantime the mechanics of this PR have been absorbed by perf itself: since Linux 5.8, The per-benchmark automation via ctest fixtures (the part that was still fighting the Closing in favor of the documentation approach. Thanks again for exploring this! |
Modern perf can generate flame graphs natively, so a short recipe in the README replaces the flame graph integration that was proposed in PR #192.
@eguiraud I tried but I am confused why it works from the terminal but doesn't generate anything in ctest...
From rootbench build directory:
and both pngs (memory and CPU flamegraphs) are beautifully generated!
But from ctest:
The command is the same for both and I see the benchmark is running actually...