[PATCH] D104129: [CSSPGO] Report zero-count probe in profile instead of dangling probes.
Wenlei He via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Jun 15 07:53:31 PDT 2021
wenlei added a comment.
> Be consistent with non-probe profile.
AutoFDO profile usually does not fill in zeros. counts below a certain threshold is omitted to save profile size.
Now in order to differentiate from unknown, the approach taken here is to always mark any known count including zero. In that sense this is different from AutoFDO (actually before this change, dangling probes are marked so zero counts can be omitted which is closer to AutoFDO for representing sparse profile).
> Improving counts quality by respecting the counts already collected on the non-dangling sibling of the danling probe
nit on the wording, sibling leads others to think it's a different probe under the same probe inline tree. Here, it's really just copies of the same probe (probes sharing the same Id).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D104129/new/
https://reviews.llvm.org/D104129
More information about the llvm-commits
mailing list