<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Mar 13, 2023 at 9:52 AM Wenlei He via Phabricator <<a href="mailto:reviews@reviews.llvm.org">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">wenlei added a comment.<br>
<br>
> There is also a small change on the create_llvm_prof tool side. We will update the tool soon.<br>
<br>
What kind of change is that about? Appreciate if you could drop us a note when the change is up since we don't closely follow create_llvm_prof updates.<br></blockquote><div><br></div><div>They are related to using the last/first entry of LBR stack to get some coverage.</div><div>Also how to handle the duplicated first lbr entry (the bogus one)</div><div>I notice you when we port the change to open source verison.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<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>
<br>
All thing being equal, I'd favor consistency. But there's potential benefit by allowing more FS profile loading, though whether that is material depends on whether we will actually leverage the ability to load more than 4 profiles. My intuition is that loading base profile + pre-RA profile + pre-layout profile as the way it is today is probably good enough.<br></blockquote><div>Also allowing more bits for each pass might also help.</div><div><br></div><div>It's possible that we add another pass for machine function splitting.</div><div> </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>
Repository:<br>
  rG LLVM Github Monorepo<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>