openoffice-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Don Lewis <truck...@apache.org>
Subject Re: branches/gbuild-reintegration merged
Date Wed, 17 Aug 2016 06:06:32 GMT
On  7 Aug, Damjan Jovanovic wrote:
> Hi
> 
> branches/gbuild-reintegration has been merged to trunk. It has > 136
> patches and makes many changes throughout the code, so you probably need to
> "dmake clean" before rebuilding.
> 
> I haven't fixed the Windows build performance regression caused by issue
> #117845, so if you're building on Windows use --disable-pch which only
> extends your build time by around 2 hours instead of 6. If you aren't happy
> with this, please help me figure out the problem.

I did some sleuthing ...

I compared build logs between --enable-pch and --disable-pch builds. The
compiler command line for the --enable-pch build has the additional
option
	-DPRECOMPILED_HEADERS
which causes precompiled_foo.hxx to pull in a whole wad of headers even
if they aren't needed to compile an individual .cxx file.  That's why
the build is taking longer.  A precompiled version of
precompiled_foo.hxx is build beforehand by passing
-Ycprecompiled_foo.hxx to the compiler, but gbuild neglects to tell the
compiler to use it when building all the .cxx files because it never
passes -Yuprecompiled_foo.hxx to the compiler.

That option is supposed to come from
gb_PrecompiledHeader_get_enableflags or
gb_NoexPrecompiledHeader_get_enableflags in LinkTarget.mk, but for some
reason that isn't getting activated.

I don't see any obvious recent changes that might have broken this.


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@openoffice.apache.org
For additional commands, e-mail: dev-help@openoffice.apache.org


Mime
View raw message