Patch to define __NO_STRING_INLINES in system.h

Zack Weinberg zack@codesourcery.com
Sun Jan 19 02:30:00 GMT 2003


"Kaveh R. Ghazi" <ghazi@caip.rutgers.edu> writes:

>  > While I am entirely in favor of this patch, I do have to point out
>  > that making just this change causes a slight, consistent increase in
>  > the text size of all the compiler binaries.  (i686-linux native build)
>  > 
>  >                     before    after   delta
>  > cc1                3775717  3777259    1542
>  > 
>  > Appended to this message is a diff of the dissassembly of alias.o.
>  > It looks like difficulty optimizing memset, but I could be wrong.
>  > zw
>
> Thanks very much for the analysis.  Chalk one up for glibc I
> guess. :-/
>
> I conclude two things, first there doesn't appear to be enough of a
> difference to warrant rejecting the patch.  (Do you agree?)

Yes.  1.5k is trivial - I've got a patch under development to knock
the text size down by ~200k.  And that's an easy one; there's more
gains to be had in the same places.

> Second, this should be fuel for someone to look at why we don't do
> quite as well as glibc.  I think Jan's done some work in that area,
> perhaps he could take a look?  (I'm certainly not qualified.)

Agree this is worth looking into.  

zw



More information about the Gcc-patches mailing list