On Wed, Jul 20, 2016 at 7:11 AM, Patrick Palka <patr...@parcs.ath.cx> wrote:
> On Wed, 20 Jul 2016, Bernd Schmidt wrote:
>
>> On 07/19/2016 10:20 AM, Richard Biener wrote:
>> > I like it. Improving re-build time in my dev tree is very much
>> > welcome, and yes,
>> > libbackend build time is a big part of it usually (plus of course cc1
>> > link time).
>>
>> Since that wasn't an entirely explicit ack, I'll add mine. Thank you for
>> doing
>> this.
>>
>>
>> Bernd
>>
>>
>
> Committed as r238524 with the following minor change to the configure
> test to use $CFLAGS and $LDFLAGS consistently:
What is interesting is I did not see any change in my bootstrap/test
times. Constant at 1 hour and 39 minutes (this includes archiving the
build away).
And yes I checked the logs to make sure T option to ar is being used too.
Thanks,
Andrew
>
> diff --git a/gcc/configure.ac b/gcc/configure.ac
> index 63052ba..241e82d 100644
> --- a/gcc/configure.ac
> +++ b/gcc/configure.ac
> @@ -4905,7 +4905,7 @@ echo 'int main (void) { return 0; }' > conftest.c
> if ($AR --version | sed 1q | grep "GNU ar" \
> && $CC $CFLAGS -c conftest.c \
> && $AR rcT conftest.a conftest.o \
> - && $CC -o conftest conftest.a) >/dev/null 2>&1; then
> + && $CC $CFLAGS $LDFLAGS -o conftest conftest.a) >/dev/null 2>&1; then
> thin_archive_support=yes
> fi
> rm -f conftest.c conftest.o conftest.a conftest
