[PATCH] D104129: [CSSPGO] Report zero-count probe in profile instead of dangling probes.

Tue Jun 15 23:38:08 PDT 2021

hoy added a comment.

In D104129#2821201 <https://reviews.llvm.org/D104129#2821201>, @wenlei wrote:

> In D104129#2821189 <https://reviews.llvm.org/D104129#2821189>, @hoy wrote:
>
>> In D104129#2821164 <https://reviews.llvm.org/D104129#2821164>, @wenlei wrote:
>>
>>>> Be consistent with dwarf line number based profile.
>>>
>>> This is not accurate, see earlier comments. What i meant was not about "non-probe profile -> dwarf line number profile".
>>>
>>> Otherwise lgtm. Thanks.
>>
>> It should be consistent with non-CS dwarf-based profile. CS dwarf-based profile doesn't have either zero or dangling reported, likely needs a fix. Is that what you meant?
>
> Non-cs dwarf-based profile normally doesn't have zero counts in the profile. Any count below a certain threshold is omitted in the profile to save size. This is the default behavior for our production usage (I believe for google as well). This is why the current change that needs explicit zero is not consistent with afdo.

I see. I'm seeing zero counts in non-CS dwarf-based profile with our internal profile generation tool like below. I guess the production profile is generated with additional switches?

main:139344172:0
 6: 0
 9: 0
 17: 0
 18: 0
 19: 0
 20: 0
 21: 0
 23: 0
 27: 0
 28: 0
 31: 0
 32: 0
 33: 0
 34: 0
 36: 0
 41: 0
 55: 0
 64: 0
 79: 0
 84: 0
 65411: 0
 33: atoi:0

  2: 0
  65175: 0

39: read_min:27098

  14: 0
  17: 0

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D104129/new/

https://reviews.llvm.org/D104129