r234143 - This reverts commit r234104, bringing back 233393 now that ARM is fixed.
rafael.espindola at gmail.com
Fri May 22 13:48:39 PDT 2015
I reverted this in r238048.
I am working on an idea to try to get most of the savings and still be
bug compatible with the current ICF implementation.
On 22 May 2015 at 16:02, Nico Weber <thakis at chromium.org> wrote:
> On Wed, May 20, 2015 at 7:41 PM, Rafael Espíndola
> <rafael.espindola at gmail.com> wrote:
>> > This regressed binary size of Chromium on Linux with 1300 kB. Our
>> > theory is that the linker  is dead-stripping more effectively with
>> > unique section names for some reason.
>> It is ICF, which is why it didn't reproduce with the clang binary.
>> I reduced this to https://sourceware.org/bugzilla/show_bug.cgi?id=18440
>> I will think about it a bit.
> Have you had a chance to think about this? Should this not be done by
> default until an ld fix is deployed in most places?
More information about the cfe-commits