Re: [PATCH 01/10] Initial import of asan from the Google branch into trunk

2012-11-12 Thread Dodji Seketeli
Tobias Burnus writes: > * -fno-address-sanitizer doesn't work (it does in Clang); it is > explicitly disabled via RejectNegative in gcc/common.opt > Fixed in common.opt by removing the RejectNegative. I am thus sending the updated patch. * common.opt: Add -faddress-sanitizer option.

Re: [PATCH 01/10] Initial import of asan from the Google branch into trunk

2012-11-09 Thread Wei Mi
> Other issues: > > * libasan does not seem to be a multilib, at least I only find the 64bit > version on x86-64-gnu-linux such that "-m32" compilation fails. > That is because originally configure file is shared between asan and tsan (tsan doesn't support 32 bit). Diego has suggested me to split

Re: [PATCH 01/10] Initial import of asan from the Google branch into trunk

2012-11-09 Thread Tobias Burnus
Jakub Jelinek wrote: On Fri, Nov 09, 2012 at 02:14:04PM +0100, Tobias Burnus wrote: I tried the 01/10 to 10/10 patch series but it doesn't trigger for the following test case: [...] Can't reproduce that (admittedly with asan SVN branch rather than the patchset): I can reproduce both; compari

Re: [PATCH 01/10] Initial import of asan from the Google branch into trunk

2012-11-09 Thread Xinliang David Li
It seems that my one line fix in asan branch (r192605) is not included in Dodji's patch set. David On Fri, Nov 9, 2012 at 5:58 AM, Jakub Jelinek wrote: > On Fri, Nov 09, 2012 at 02:14:04PM +0100, Tobias Burnus wrote: >> Dodji Seketeli wrote: >> >This patch imports the initial state of asan as i

Re: [PATCH 01/10] Initial import of asan from the Google branch into trunk

2012-11-09 Thread Jakub Jelinek
On Fri, Nov 09, 2012 at 02:14:04PM +0100, Tobias Burnus wrote: > Dodji Seketeli wrote: > >This patch imports the initial state of asan as it was in the > >Google branch. > > > >It provides basic infrastructure for asan to instrument memory > >accesses on the heap, at -O3. Note that it supports nei

Re: [PATCH 01/10] Initial import of asan from the Google branch into trunk

2012-11-09 Thread Tobias Burnus
Dodji Seketeli wrote: This patch imports the initial state of asan as it was in the Google branch. It provides basic infrastructure for asan to instrument memory accesses on the heap, at -O3. Note that it supports neither stack nor global variable protection. I tried the 01/10 to 10/10 patch

Re: [PATCH 01/10] Initial import of asan from the Google branch into trunk

2012-11-06 Thread Diego Novillo
On 2012-11-02 15:56 , Dodji Seketeli wrote: This patch imports the initial state of asan as it was in the Google branch. It provides basic infrastructure for asan to instrument memory accesses on the heap, at -O3. Note that it supports neither stack nor global variable protection. The rest of

[PATCH 01/10] Initial import of asan from the Google branch into trunk

2012-11-02 Thread Dodji Seketeli
This patch imports the initial state of asan as it was in the Google branch. It provides basic infrastructure for asan to instrument memory accesses on the heap, at -O3. Note that it supports neither stack nor global variable protection. The rest of the patches of the set is intended to further