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

Amir Ayupov via llvm-commits llvm-commits at lists.llvm.org
Fri May 23 10:12:53 PDT 2025


aaupov wrote:

> Instead of introducing the new option, can we reuse `--block-size=` and make it accept multiple values or a different format?
> 
> > If rescaling is impossible, no heatmap is produced.
> 
> Without a printed warning? I understand that the size limitation is driven by implementation, but at the very least we should report that there is no expected output.
> 
> Alternatively, use a different option format that encapsulates the limitations. E.g., `--block-size=<initial_size>{:<scale>:<count>}` or `--block-size=<initial_size>{:<scale1>,<scale2>...}`.

Good call. I prefer the latter approach with explicit scales.


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


More information about the llvm-commits mailing list