[PATCH] D109713: [AutoFDO][llvm-profgen] Report zero count for unexecuted part of function code
Lei Wang via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Sep 14 16:56:54 PDT 2021
wlei added a comment.
In D109713#3000685 <https://reviews.llvm.org/D109713#3000685>, @hoy wrote:
> Thanks for the change. I'm wondering instead of extending `findDisjointRanges` if we can just add a post-processing pass that adds a zero sample to all line offsets. We did this for probe-based CS profile.
The advantage to do it by extending `findDisjointRanges` is it can save some operations and adding zero sample for line-number is heavier than probe because it need to travels the inline callstack.
For extreme case say if we have counter like [1,9999] : 1. and the function range is [1,10000], post-processing will do both for [1,9999] : 1, [1,10000]: 0 . it's almost double the cost. pre-processing will do [1,9999] : 1 and [10000,10000]:0.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D109713/new/
https://reviews.llvm.org/D109713
More information about the llvm-commits
mailing list