<div dir="ltr"><div>LGTM with a few nits. I uploaded it to <a href="http://reviews.llvm.org/D16690">http://reviews.llvm.org/D16690</a> for me. Can you address the comments I left there?</div><div><br></div><div>As to speed, that numbers are fantastic. I originally thought 4% might be too large, but if it is 0.58% or 0.1%, it's no brainer. And it is interesting that you got the exact same strtab sizes, because it means that there are absolutely no global or local symbols sharing the same even in fairly large programs.</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Jan 28, 2016 at 8:45 AM, Rafael Espíndola <span dir="ltr"><<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">>> When linking clang on a Opteron this goes from 1.090608063 to<br>
>> 1.135082244 seconds (1.04X slower)<br>
>><br>
>> and strtab goes from 5426172 to 5319595 bytes (1.02X smaller).<br>
>><br>
>> Linking scylladb goes from 8.104377098 to 8.116767958 (1.0015X slower)<br>
>> and the strtab goes from 14824893 to 14443221 (1.026X smaller).<br>
>><br>
><br>
> The savings are relatively small, but still greater than zero. For a<br>
> small increase in cost, at least for scylladb.<br>
> For clang the link time increases a little bit more but I still feel<br>
> that tolerable. We can always re-evaluate this in the future if it<br>
> shows up being a problem.<br>
<br>
</span>Yesterday Rui remarked that global symbol names are already uniqued in<br>
the symbol table. I first thought about reusing the hash, but then<br>
decided to try something even simpler: don't unique strings from the<br>
symbol table. This means we would duplicate a string if a local symbol<br>
has the same name as a global, but merge other cases.<br>
<br>
In both clang and scylladb I get the same exact same strtab sizes<br>
(1.02X smaller and 1.026X smaller), but now the slowdowns are just<br>
1.0058X and 1.00106X.<br>
<br>
Cheers,<br>
Rafael<br>
<br>_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
<br></blockquote></div><br></div>