r234143 - This reverts commit r234104, bringing back 233393 now that ARM is fixed.

Hans Wennborg hans at chromium.org
Thu May 14 13:22:02 PDT 2015


Hi Rafael,

Sorry for following up so late after-the-fact.

On Sun, Apr 5, 2015 at 9:36 PM, Rafael Espindola
<rafael.espindola at gmail.com> wrote:
> Author: rafael
> Date: Sun Apr  5 23:36:45 2015
> New Revision: 234143
>
> URL: http://llvm.org/viewvc/llvm-project?rev=234143&view=rev
> Log:
> This reverts commit r234104, bringing back  233393 now that ARM is fixed.
>
> Original message:
>
> Don't use unique section names by default if using the integrated as.
>
> This saves some IO and ccache space by not creating long section names. It
> should work with every ELF linker.

This regressed binary size of Chromium on Linux with 1300 kB. Our
theory is that the linker [1] is dead-stripping more effectively with
unique section names for some reason.

We can work around this with -funique-section-names, but maybe we
should hold back on this change until the linker issue is understood?

 - Hans

 [1] We use GNU gold (GNU Binutils 2.24) 1.11



More information about the cfe-commits mailing list