[llvm] [BOLT] Fix heatmaps on large BOLTEd binaries. (PR #92815)

Paschalis Mpeis via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 28 03:39:31 PDT 2024


paschalis-mpeis wrote:


Hi @maksfb, thanks for your review.

I went with a unit test. Consider this as a starting point and feel free to suggest more changes.

---

In short:
I added a flag `perf-script-events` to bypass BOLT's 'perf script' processing,
and I use it to supply custom mmap event entries that demonstrate the problem.

I invoke `preprocessProfile` with the mock perf profile+binary and finally check the computed size. I needed to expose a const reference to `BinaryMMapInfo` for checking the size.

The unit test ignores some errors incurred by not supplying valid 'memory events' for irrelevant processing in `preprocessProfile`.


https://github.com/llvm/llvm-project/pull/92815


More information about the llvm-commits mailing list