Il 20/03/2014 13:24, Jakub Jelinek ha scritto:
Hi!

We currently bootstrap both libsanitizer and libvtv, the former
just in case somebody decides to --with-build-config=bootstrap-asan
(or --with-build-config=bootstrap-ubsan), the latter if somebody decides
to --enable-vtable-verify.

This patch changes it so that we only bootstrap libsanitizer if
bootstrap-asan or bootstrap-ubsan is in BOOT_CONFIG, and only bootstrap
libvtv if --enable-vtable-verify.

Bootstrapped/regtested on x86_64-linux and i686-linux, on x86_64 it
saved a few minutes of build time, neither libsanitizer nor libvtv
has been bootstrapped, only built in stage3.
Currently also bootstrapping on i686-linux with
--enable-vtable-verify --with-build-config=bootstrap-asan
but already got into stage2 and verified that both libsanitizer and libvtv
are being bootstrapped in that case.

Ok for trunk?

2014-03-20  Jakub Jelinek  <ja...@redhat.com>

        * configure.ac: Move BUILD_CONFIG set up earlier.  Add
        --enable-vtable-verify option parsing.  Don't add
        target-libsanitizer to bootstrap_target_libs unless
        --with-build-config=bootstrap-asan or
        --with-build-config=bootstrap-ubsan.  Don't add target-libvtv
        to bootstrap_target_libs unless --enable-vtable-verify.
        * configure: Regenerated.

Is the extra complication really worth saving a few minutes of compile time?

Paolo

Reply via email to