Re: PATCH: Remove AM_MAKEFLAGS from libsanitizer

2012-12-17 Thread H.J. Lu
On Wed, Dec 12, 2012 at 11:32 PM, Paolo Bonzini wrote: > Il 13/12/2012 00:23, H.J. Lu ha scritto: >> On Wed, Dec 12, 2012 at 3:01 PM, H.J. Lu wrote: >>> On Wed, Dec 12, 2012 at 2:13 PM, Paolo Bonzini wrote: Il 12/12/2012 19:11, H.J. Lu ha scritto: > > in RAW_CXX_TARGET_EXPOR

Re: PATCH: Remove AM_MAKEFLAGS from libsanitizer

2012-12-12 Thread Paolo Bonzini
Il 13/12/2012 00:23, H.J. Lu ha scritto: > On Wed, Dec 12, 2012 at 3:01 PM, H.J. Lu wrote: >> On Wed, Dec 12, 2012 at 2:13 PM, Paolo Bonzini wrote: >>> Il 12/12/2012 19:11, H.J. Lu ha scritto: in RAW_CXX_TARGET_EXPORTS. There is no need to do anything. >> >> Nope, if yo

Re: PATCH: Remove AM_MAKEFLAGS from libsanitizer

2012-12-12 Thread H.J. Lu
On Wed, Dec 12, 2012 at 3:01 PM, H.J. Lu wrote: > On Wed, Dec 12, 2012 at 2:13 PM, Paolo Bonzini wrote: >> Il 12/12/2012 19:11, H.J. Lu ha scritto: > >> > >> in RAW_CXX_TARGET_EXPORTS. There is no need to do anything. > > Nope, if you remove this you get the wrong definition of

Re: PATCH: Remove AM_MAKEFLAGS from libsanitizer

2012-12-12 Thread H.J. Lu
On Wed, Dec 12, 2012 at 2:13 PM, Paolo Bonzini wrote: > Il 12/12/2012 19:11, H.J. Lu ha scritto: >> >> in RAW_CXX_TARGET_EXPORTS. There is no need to do anything. >>> > >>> > Nope, if you remove this you get the wrong definition of CC and CXX from >>> > EXTRA_TARGET_FLAGS. Instead, you

Re: PATCH: Remove AM_MAKEFLAGS from libsanitizer

2012-12-12 Thread Paolo Bonzini
Il 12/12/2012 19:11, H.J. Lu ha scritto: >>> >> >>> >> in RAW_CXX_TARGET_EXPORTS. There is no need to do anything. >> > >> > Nope, if you remove this you get the wrong definition of CC and CXX from >> > EXTRA_TARGET_FLAGS. Instead, you need to add RAW_CXX_FOR_TARGET to >> > AM_MAKEFLAGS and EXTRA

Re: PATCH: Remove AM_MAKEFLAGS from libsanitizer

2012-12-12 Thread H.J. Lu
On Wed, Dec 12, 2012 at 10:01 AM, Paolo Bonzini wrote: > Il 12/12/2012 18:30, H.J. Lu ha scritto: >> On Wed, Dec 12, 2012 at 6:46 AM, Paolo Bonzini wrote: >>> Il 12/12/2012 15:41, H.J. Lu ha scritto: MAKEOVERRIDES is used for multilib. I got /bin/sh ../libtool --tag=CXX --mode=c

Re: PATCH: Remove AM_MAKEFLAGS from libsanitizer

2012-12-12 Thread Paolo Bonzini
Il 12/12/2012 18:30, H.J. Lu ha scritto: > On Wed, Dec 12, 2012 at 6:46 AM, Paolo Bonzini wrote: >> Il 12/12/2012 15:41, H.J. Lu ha scritto: >>> MAKEOVERRIDES is used for multilib. I got >>> >>> /bin/sh ../libtool --tag=CXX --mode=compile -D_GNU_SOURCE -D_DEBUG >>> -D__STDC_CONSTANT_MACROS -D_

Re: PATCH: Remove AM_MAKEFLAGS from libsanitizer

2012-12-12 Thread H.J. Lu
On Wed, Dec 12, 2012 at 6:46 AM, Paolo Bonzini wrote: > Il 12/12/2012 15:41, H.J. Lu ha scritto: >> MAKEOVERRIDES is used for multilib. I got >> >> /bin/sh ../libtool --tag=CXX --mode=compile -D_GNU_SOURCE -D_DEBUG >> -D__STDC_CONSTANT_MACROS -D__STDC_FORMAT_MACROS -D__STDC_LIMIT_MACROS >> -DA

Re: PATCH: Remove AM_MAKEFLAGS from libsanitizer

2012-12-12 Thread Paolo Bonzini
Il 12/12/2012 15:41, H.J. Lu ha scritto: > MAKEOVERRIDES is used for multilib. I got > > /bin/sh ../libtool --tag=CXX --mode=compile -D_GNU_SOURCE -D_DEBUG > -D__STDC_CONSTANT_MACROS -D__STDC_FORMAT_MACROS -D__STDC_LIMIT_MACROS > -DASAN_HAS_EXCEPTIONS=1 -DASAN_FLEXIBLE_MAPPING_AND_OFFSET=0 > -

Re: PATCH: Remove AM_MAKEFLAGS from libsanitizer

2012-12-12 Thread H.J. Lu
On Wed, Dec 12, 2012 at 12:49 AM, Paolo Bonzini wrote: > Il 11/12/2012 22:39, H.J. Lu ha scritto: >> On Tue, Dec 11, 2012 at 6:36 AM, Paolo Bonzini wrote: >>> 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.

Re: PATCH: Remove AM_MAKEFLAGS from libsanitizer

2012-12-12 Thread Paolo Bonzini
Il 11/12/2012 22:39, H.J. Lu ha scritto: > On Tue, Dec 11, 2012 at 6:36 AM, Paolo Bonzini wrote: >> 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 f

PATCH: Remove AM_MAKEFLAGS from libsanitizer

2012-12-11 Thread H.J. Lu
On Tue, Dec 11, 2012 at 6:36 AM, Paolo Bonzini wrote: > 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 n