[PATCH] D140063: [AutoFDO] Use getHeadSamplesEstimate instead of getTotalSamples to compute profile callsite staleness
Wenlei He via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Dec 15 09:56:34 PST 2022
wenlei added a comment.
I thought the idea is to compute the % of samples being dropped due to mismatch? in this case, all samples from callsite will be dropped, so I actually don't see a problem with using getTotalSamples. Yes you can argue that they can be merged with base profile, but then that argument applies to getHeadSamplesEstimate too?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D140063/new/
https://reviews.llvm.org/D140063
More information about the llvm-commits
mailing list