[PATCH] D147456: [AutoFDO] Stale profile matching(part 1)
Lei Wang via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Apr 5 18:04:36 PDT 2023
wlei added a comment.
In D147456#4247167 <https://reviews.llvm.org/D147456#4247167>, @mingmingl wrote:
> In D147456#4247075 <https://reviews.llvm.org/D147456#4247075>, @wlei wrote:
>
>> Thank you for all the feedback, I will try to address it later. I saw you all are curious the algorithm part, here it is: https://reviews.llvm.org/D147545.
>>
>>> Just browsed the patch, I don't seem to find how FUncToMatchingsMap is populated which is the core of the stale profile matching algorithm. Does algorithm use some anchor points to do the remapping?
>>
>> Yes, here is the algorithm part : https://reviews.llvm.org/D147545, it leverages the callee name/ calltarget name as anchor which doesn't need any other additional info.
>
> Just curious, I wonder if there //happens// to be a measurement of the percentage of samples recovered (that would have been dropped) for csspgo. I guess I'm trying see how much headroom remains with D147545 <https://reviews.llvm.org/D147545> :)
Good point, so far no, it's doable, we have the https://reviews.llvm.org/D136627 to compute how many samples are dropped due to the CSSPGO's checksum mismatch, which is what this stale profile matching want to reuse. Similarly, after the matching in D147545 <https://reviews.llvm.org/D147545>, we can compute how many samples/callsites in profile still remains mismatched.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D147456/new/
https://reviews.llvm.org/D147456
More information about the llvm-commits
mailing list