<div dir="ltr"><div dir="ltr"><div class="gmail_default" style="font-family:monospace;font-size:small;color:#000000"><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sun, Feb 21, 2021 at 2:36 PM Petr Hosek 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">phosek added a comment.<br>
<br>
I plan on improving how `__llvm_prf_names` is constructed using the approach suggested in PR49155 in a follow up change.<br></blockquote><div><br></div><div><br></div><div><div class="gmail_default" style="font-family:monospace;font-size:small;color:rgb(0,0,0)">Compression can reduce per object size and therefore the size of total linker input. </div><div class="gmail_default" style="font-family:monospace;font-size:small;color:rgb(0,0,0)"><br></div><div class="gmail_default" style="font-family:monospace;font-size:small;color:rgb(0,0,0)">If the size of name section in the final binary is the concern, we can make the name section non loadable (like coverage map) then it can be stripped from the binary and put into its own file. The llvm_profdata tool will need to read this file in order to produce the index format because names won't available in the raw profile any more.  The downside of this approach is the reduced usability.</div><div class="gmail_default" style="font-family:monospace;font-size:small;color:rgb(0,0,0)"><br></div><div class="gmail_default" style="font-family:monospace;font-size:small;color:rgb(0,0,0)">David</div><div class="gmail_default" style="font-family:monospace;font-size:small;color:rgb(0,0,0)"><br></div></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/D96757/new/" rel="noreferrer" target="_blank">https://reviews.llvm.org/D96757/new/</a><br>
<br>
<a href="https://reviews.llvm.org/D96757" rel="noreferrer" target="_blank">https://reviews.llvm.org/D96757</a><br>
<br>
</blockquote></div></div>