<div dir="ltr"><div dir="ltr"><br></div><div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Mar 9, 2023 at 10:21 AM Hongtao Yu via Phabricator <<a href="mailto:reviews@reviews.llvm.org" target="_blank">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">hoy added a comment.<br>
<br>
In D145171#4182014 <<a href="https://reviews.llvm.org/D145171#4182014" rel="noreferrer" target="_blank">https://reviews.llvm.org/D145171#4182014</a>>, @xur wrote:<br>
<br>
> The total improvement really depends on programs. On average, I would say 1.5% to 2.0%.  For some programs, like clang itself, FSAFDO improve ~3% over AFDO.<br>
> There is also a small change on the create_llvm_prof tool side. We will update the tool soon.<br>
<br>
The numbers are exciting! I'll let you know our numbers once ready.<br>
<br>
BTW, I'll started the integration with CSSPGO base off here. Pseudo probes have a potential to allow more bits for the FS discriminator and additional FS passes. Implementation-wise, the potential comes from placing the FS discriminator field as a separate int64 operand of the pseudo probe intrinsic, instead of using the existing dbg metadata. I would like your thoughts about whether it's worth the diversion. Thanks.<br></blockquote><div>Interesting. Having more bits for different rounds definitely helps -- we can encode more information to detect mismatch or do a partial match. Looking forward to seeing your CSSPGO work in this regard.</div><div><br></div><div>In Google, we are also considering extending the discriminator in dbg info. The DWARF standards specified DW_LNE_set_discriminator to take an ULEB128 integer. So in theory we can use arbitrarily large integer types. Extending the discriminator to 64 bits in LLVM is doable.  </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
<br>
CHANGES SINCE LAST ACTION<br>
  <a href="https://reviews.llvm.org/D145171/new/" rel="noreferrer" target="_blank">https://reviews.llvm.org/D145171/new/</a><br>
<br>
<a href="https://reviews.llvm.org/D145171" rel="noreferrer" target="_blank">https://reviews.llvm.org/D145171</a><br>
<br>
</blockquote></div></div>
</div>