[llvm] [BOLT] Support profile density with basic samples (PR #137644)
Maksim Panchenko via llvm-commits
llvm-commits at lists.llvm.org
Fri May 9 16:37:51 PDT 2025
maksfb wrote:
> > members that used to have "branch" in the name now use "sample" which means "sample or branch", which makes "sample" ambiguous.
>
> "Sample" is supposed to cover different profile types. We disambiguate it by adding "basic" or "branch" to it. So I'd say this change generalizes stats that were only applicable to branch samples to cover basic samples as well. Do you agree?
In general, outside of context of `DataAggregator`, I agree. But in the code, we refer to non-LBR samples as just samples.
https://github.com/llvm/llvm-project/pull/137644
More information about the llvm-commits
mailing list