[llvm] [BOLT][heatmap] Produce zoomed-out heatmaps (PR #140153)

Maksim Panchenko via llvm-commits llvm-commits at lists.llvm.org
Mon May 26 22:28:15 PDT 2025


https://github.com/maksfb commented:

The implementation looks good to me.

A couple comments regarding the interface:
  * If we are producing multiple files by default, we might have to change docs in more places.
  * I find the power-of-two limitation for the scale unnecessary.
  * The variance in default scale steps looks a bit unnatural (64, 4, 64 again). I would keep just 64 and 64 making the default sizes 64 bytes, 4 KB, 256 KB.

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


More information about the llvm-commits mailing list