<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Mon, Jan 18, 2016 at 12:13 PM, Rafael Espíndola <span dir="ltr"><<a href="mailto:rafael.espindola@gmail.com" target="_blank">rafael.espindola@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">>><br>
>> r257017:<br>
>> * strtab is 3916859 bytes<br>
>> * link takes 0.317558552 seconds<br>
>><br>
>> r257016:<br>
>> * strtab is 3812950 bytes (1.027* smaller)<br>
>> * link takes 0.353217270 seconds (1.11* slower)<br>
>><br>
>> r257016 with StrTabBuilder.finalize commented out as a timing hack.<br>
>> * link time is 0.331792363 (1.04X slower)<br>
</span>...<br>
<span class=""><br>
><br>
> What's the string table size when you disabled tail merging but did<br>
> de-duplication?<br>
<br>
</span>It might be 3813075 bytes. I say might because I haven't finished the<br>
patch and might have missed something.<br></blockquote><div><br></div><div>You can just hack up StringTableBuilder so that they don't add the same string more than once, then the resulting string table contains no duplicates, no? </div></div></div></div>