Re: [PATCH] Support -fsanitize=leak

2013-11-22 Thread Jakub Jelinek
On Fri, Nov 22, 2013 at 09:47:13PM +0100, Dodji Seketeli wrote: > > This patch adds support for -fsanitize=leak and -static-liblsan options. > > If combined with -fsanitize=address, it does nothing, > > >From this hunk: > > @@ -8123,7 +8133,10 @@ sanitize_spec_function (int argc, const >

Re: [PATCH] Support -fsanitize=leak

2013-11-22 Thread Dodji Seketeli
Jakub Jelinek writes: > This patch adds support for -fsanitize=leak and -static-liblsan options. > If combined with -fsanitize=address, it does nothing, >From this hunk: @@ -8123,7 +8133,10 @@ sanitize_spec_function (int argc, const return (flag_sanitize & SANITIZE_THREAD) ? "" : N

Re: [PATCH] Support -fsanitize=leak

2013-11-20 Thread Jakub Jelinek
On Wed, Nov 20, 2013 at 11:40:14AM +0400, Yury Gribov wrote: > > Jakub already wrote the invoke.texi part: > > http://gcc.gnu.org/ml/gcc-patches/2013-11/msg02061.html > > Ah, thanks, I somehow missed it. When will this end up in trunk? I > have a depending patch. When it is reviewed. I have vari

Re: [PATCH] Support -fsanitize=leak

2013-11-19 Thread Yury Gribov
> Jakub already wrote the invoke.texi part: > http://gcc.gnu.org/ml/gcc-patches/2013-11/msg02061.html Ah, thanks, I somehow missed it. When will this end up in trunk? I have a depending patch. -Y

Re: [PATCH] Support -fsanitize=leak

2013-11-19 Thread Marek Polacek
On Wed, Nov 20, 2013 at 09:28:48AM +0400, Yury Gribov wrote: > >>This patch adds support for -fsanitize=leak and -static-liblsan options. > >>... > > > >Documentation? > > > >Ian > > In case this is a prereq for commit, here is a simple addition to invoke.texi. Jakub already wrote the invoke.texi

Re: [PATCH] Support -fsanitize=leak

2013-11-19 Thread Yury Gribov
This patch adds support for -fsanitize=leak and -static-liblsan options. ... Documentation? Ian In case this is a prereq for commit, here is a simple addition to invoke.texi. -Y diff --git a/gcc/doc/invoke.texi b/gcc/doc/invoke.texi index 7074a48..960df85 100644 --- a/gcc/doc/invoke.texi ++

Re: [PATCH] Support -fsanitize=leak

2013-11-18 Thread Marek Polacek
On Mon, Nov 18, 2013 at 01:15:02PM +0100, Marek Polacek wrote: > Moreover, it fixes the bootstrap-ubsan failure when > building fixincl - I guess the gnu-user.h hunk was missing. Thanks. Sorry, turned out this is bogus. It really will need some configure/Makefile fix. Marek

Re: [PATCH] Support -fsanitize=leak

2013-11-18 Thread Jakub Jelinek
On Fri, Nov 15, 2013 at 12:34:14PM -0800, Ian Lance Taylor wrote: > On Fri, Nov 15, 2013 at 11:41 AM, Jakub Jelinek wrote: > > > > This patch adds support for -fsanitize=leak and -static-liblsan options. > > If combined with -fsanitize=address, it does nothing, otherwise it links > > in liblsan, a

Re: [PATCH] Support -fsanitize=leak

2013-11-18 Thread Marek Polacek
On Fri, Nov 15, 2013 at 08:41:38PM +0100, Jakub Jelinek wrote: > Hi! > > This patch adds support for -fsanitize=leak and -static-liblsan options. > If combined with -fsanitize=address, it does nothing, otherwise it links > in liblsan, a new shared+static library (on x86_64-linux only so far, > the

Re: [PATCH] Support -fsanitize=leak

2013-11-15 Thread Ian Lance Taylor
On Fri, Nov 15, 2013 at 11:41 AM, Jakub Jelinek wrote: > > This patch adds support for -fsanitize=leak and -static-liblsan options. > If combined with -fsanitize=address, it does nothing, otherwise it links > in liblsan, a new shared+static library (on x86_64-linux only so far, > the code isn't 32

[PATCH] Support -fsanitize=leak

2013-11-15 Thread Jakub Jelinek
Hi! This patch adds support for -fsanitize=leak and -static-liblsan options. If combined with -fsanitize=address, it does nothing, otherwise it links in liblsan, a new shared+static library (on x86_64-linux only so far, the code isn't 32-bit ready apparently). Bootstrapped/regtested on x86_64-lin