[Libreoffice] -Wl,-O1 (was Reducing -ggdb3 to -ggdb2)

On Tuesday 12 of April 2011, Caolán McNamara wrote:
> On Tue, 2011-04-12 at 16:01 +0200, Lubos Lunak wrote:
> > it. Almost five times the size of data is really a lot more data for I/O
> > to shuffle back and forth.
>
> I noticed recently that my linux --eanble-symbols solver is about 23gigs
> in size, if it was e.g. half that I could play around with putting the
> whole solver into tmpfs/ramdisk. So sounds good to me.
>
> >  Also, when linking, I see sometimes -Wl,-O1 used, but only for some
> > libraries. Does anybody know where that comes from?
>
> solenv/inc/unxgcc.mk:LINKFLAGSOPTIMIZE*=-Wl,-O1

 No, I forgot to mention, that's not the one:

R=/home/llunak/build/src && S=$R/libo && O=$S/solver/300/unxlngx6.pro && 
W=$O/workdir &&  mkdir -p $W/LinkTarget/Library/ && 
g++ -shared -Wl,-z,noexecstack '-Wl,-rpath,$ORIGIN:
$ORIGIN/../ure-link/lib' '-Wl,-rpath-link,
$O/lib' -Wl,-z,defs  -Wl,-rpath-link,/lib:/usr/lib -Wl,-z,combreloc   -L$S/solenv/unxlngx6/lib 
-L$O/lib -L$S/solenv/unxlngx6/lib -L/usr/lib64/xulrunner-2.0  -Wl,--hash-style=gnu  
-Wl,--dynamic-list-cpp-new -Wl,--dynamic-list-cpp-typeinfo -Wl,-Bsymbolic-functions  -Wl,-O1 
-Wl,-z,noexecstack   
$W/CxxObject/sw/source/ui/vba/service.o ...
$W/CxxObject/sw/source/ui/vba/wordvbahelper.o   -Wl,--start-group  -Wl,--end-group -lcomphelpgcc3 
-luno_cppu -luno_cppuhelpergcc3 -leditenglx -li18nisolang1gcc3 -luno_sal -lsblx -lsfxlx -lsvllx 
-lsvtlx -lsvxlx -lsvxcorelx -lswlx -ltklx -ltllx -lutllx -lvbahelperlx -lvcllx -o 
$W/LinkTarget/Library/vbaswobj.uno.so

 Not sure how this long line will end up looking in a mail, but this is for 
vbaswobj.uno.so and the important part 
is "-Wl,-Bsymbolic-functions  -Wl,-O1 -Wl,-z,noexecstack". Note that it does 
not include the "-Wl,--version-script" from the next line which seems to be 
the only place where LINKFLAGSOPTIMIZE is used. And I also changed 
LINKFLAGSOPTIMIZE itself for the build and it doesn't have any effect, so I'm 
sure it's not that one.

> presumably it should also be in there if it was considered good enough
> to have in the older dmake one.

 -Wl,-O1 is presumably good for final builds (i.e. what is shipped to people), 
but not for developers, as I expect the increased link time is not worth the 
somewhat shorter load time.

-- 
 Lubos Lunak
 l.lunak@suse.cz