Re: PING^2: [PATCH] PR sanitizer/55533: Can't bootstrap libsanitizer

2012-12-11 Thread Paolo Bonzini
Il 11/12/2012 20:27, H.J. Lu ha scritto: > On Tue, Dec 11, 2012 at 6:36 AM, Paolo Bonzini wrote: > >> >> As a followup please check if AM_MAKEFLAGS is needed at all. >> > diff --git a/libsanitizer/Makefile.in b/libsanitizer/Makefile.in > index 21c2711..53e0be9 100644 > --- a/libsaniti

Re: PING^2: [PATCH] PR sanitizer/55533: Can't bootstrap libsanitizer

2012-12-11 Thread H.J. Lu
On Tue, Dec 11, 2012 at 6:36 AM, Paolo Bonzini wrote: > > As a followup please check if AM_MAKEFLAGS is needed at all. > diff --git a/libsanitizer/Makefile.in b/libsanitizer/Makefile.in index 21c2711..53e0be9 100644 --- a/libsanitizer/Makefile.in +++ b/libsanitizer/Makefile.in

Re: PING^2: [PATCH] PR sanitizer/55533: Can't bootstrap libsanitizer

2012-12-11 Thread Paolo Bonzini
Il 11/12/2012 17:55, H.J. Lu ha scritto: >> > Does it use any GCC toplevel files except when regenerating the >> > configury? But I think it's a useful refactoring anyway. > libsanitizer started as standalone and it didn't work with GCC build. > We removed all those extra files to get it compile a

Re: PING^2: [PATCH] PR sanitizer/55533: Can't bootstrap libsanitizer

2012-12-11 Thread H.J. Lu
On Tue, Dec 11, 2012 at 8:19 AM, Paolo Bonzini wrote: > Il 11/12/2012 16:03, H.J. Lu ha scritto: >> diff --git a/libsanitizer/asan/Makefile.am >> b/libsanitizer/asan/Makefile.am >> index 3da1db3..45fb3b3 100644 >> --- a/libsanitizer/asan/Makefile.am >> +++ b/libsanitizer/asan/

Re: PING^2: [PATCH] PR sanitizer/55533: Can't bootstrap libsanitizer

2012-12-11 Thread Paolo Bonzini
Il 11/12/2012 16:03, H.J. Lu ha scritto: > diff --git a/libsanitizer/asan/Makefile.am b/libsanitizer/asan/Makefile.am > index 3da1db3..45fb3b3 100644 > --- a/libsanitizer/asan/Makefile.am > +++ b/libsanitizer/asan/Makefile.am > @@ -5,6 +5,10 @@ gcc_version := $(shell cat $(top_s

Re: PING^2: [PATCH] PR sanitizer/55533: Can't bootstrap libsanitizer

2012-12-11 Thread H.J. Lu
On Tue, Dec 11, 2012 at 6:36 AM, Paolo Bonzini wrote: > As a followup please check if AM_MAKEFLAGS is needed at all. I think it is just copied from elsewhere. I will take a look after this. diff --git a/libsanitizer/Makefile.in b/libsanitizer/Makefile.in index 21c2711..53e0be9 100644

Re: PING^2: [PATCH] PR sanitizer/55533: Can't bootstrap libsanitizer

2012-12-11 Thread Paolo Bonzini
Il 11/12/2012 14:47, H.J. Lu ha scritto: > On Thu, Dec 6, 2012 at 7:07 AM, H.J. Lu wrote: >> On Thu, Nov 29, 2012 at 10:30 AM, H.J. Lu wrote: >>> Hi, >>> >>> Since libsanitizer is used for bootstrap and compiled with raw_cxx, >>> we need to use explicit -I for libstdc++-v3 header files in >>> lib

PING^2: [PATCH] PR sanitizer/55533: Can't bootstrap libsanitizer

2012-12-11 Thread H.J. Lu
On Thu, Dec 6, 2012 at 7:07 AM, H.J. Lu wrote: > On Thu, Nov 29, 2012 at 10:30 AM, H.J. Lu wrote: >> Hi, >> >> Since libsanitizer is used for bootstrap and compiled with raw_cxx, >> we need to use explicit -I for libstdc++-v3 header files in >> libsanitizer. Otherwise, we will get >> >> libtool: