This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] Add --with-build-config=bootstrap-asan support
- From: Tobias Burnus <burnus at net-b dot de>
- To: "H.J. Lu" <hjl dot tools at gmail dot com>
- Cc: gcc-patches at gcc dot gnu dot org
- Date: Thu, 29 Nov 2012 19:03:06 +0100
- Subject: Re: [PATCH] Add --with-build-config=bootstrap-asan support
- References: <20121122203747.GA4264@gmail.com> <50B79D18.6050601@net-b.de>
H.J. Lu wrote:
Please try this: http://gcc.gnu.org/ml/gcc-patches/2012-11/msg01530.html
I just did locally "git log -2 -p|patch -p1 -R" - and wait until you
sort that out with the patch reviewers, which seemingly didn't like that
patch.
BTW: I am not sure whether it is possible with the current build system,
but I'd prefer: if libsanitizer is not needed (i.e. no bootstrap-asan is
used), it is only build as target library in stage2/stage3 and not also
in stage1 as host library.
Tobias Burnus wrote:
I think that patch has broken bootstrap for me. If I do a normal
bootstrap, Stage1 fails with:
I should have quoted more. Note that the "g++" or similar is missing
before "-D_GNU_SOURCE":
/bin/sh ../libtool --tag=CXX --mode=compile -D_GNU_SOURCE -D_DEBUG
-D__STDC_CONSTANT_MACROS -D__STDC_FORMAT_MACROS -D__STDC_LIMIT_MACROS
-I. -I../../../../libsanitizer/interception -I
../../../../libsanitizer/include -Wall -W -Wno-unused-parameter
-Wwrite-strings -pedantic -Wno-long-long -fPIC -fno-builtin
-fno-exceptions -fomit-frame-pointer -funwind-tables -fvisibility=hidden
-Wno-variadic-macros -Wno-c99-extensions -g -O2 -D_GNU_SOURCE -MT
interception_mac.lo -MD -MP -MF .deps/interception_mac.Tpo -c -o
interception_mac.lo
../../../../libsanitizer/interception/interception_mac.cc
Thus, for libsanitizer, the $(CXX) variable is not set, when invoked via
build root's Makefile. Which gets set via 'CXX=$$(RAW_CXX_FOR_TARGET)'
Tobias