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