[llvm] MachineBlockPlacement: Add tolerance to comparisons (PR #67197)
via llvm-commits
llvm-commits at lists.llvm.org
Tue Sep 26 13:12:40 PDT 2023
WenleiHe wrote:
> I'm not sure I can sensible measure it with our production workloads as BOLT will reorder all the basic blocks code again anyway for us
We can do prod measurement with BOLT turned off. I don't expect meaningful perf movement, but it's indeed a good sanity check.
> I could also see this being useful to reduce day-to-day churn as sampling PGO data changed, though that likely needs tuning and tighter values for the BlockFreqCompBits constant added here.
Yes. It needs to be 1) off by default initially, 2) tunable for the rounding. And just to be clear, I'm mostly thinking about rounding within BFI, not this change as is.
https://github.com/llvm/llvm-project/pull/67197
More information about the llvm-commits
mailing list