Re: [PATCH] Handle -fsanitize=leak more similarly to address/thread

2014-08-29 Thread Jeff Law
On 08/14/14 10:37, Jakub Jelinek wrote: Hi! Right now when -fsanitize=leak adds -llsan, it adds it late on the command line, so e.g. -lstdc++ comes after it, which seems to be bad. The following patch puts it early on the link command line like we do for -lasan or -ltsan. Bootstrapped/regtested

Re: [PATCH] Handle -fsanitize=leak more similarly to address/thread

2014-08-17 Thread Dodji Seketeli
Jakub Jelinek writes: > Right now when -fsanitize=leak adds -llsan, it adds it late on the command > line, so e.g. -lstdc++ comes after it, which seems to be bad. > The following patch puts it early on the link command line like we do for > -lasan or -ltsan. Bootstrapped/regtested on x86_64-lin

[PATCH] Handle -fsanitize=leak more similarly to address/thread

2014-08-14 Thread Jakub Jelinek
Hi! Right now when -fsanitize=leak adds -llsan, it adds it late on the command line, so e.g. -lstdc++ comes after it, which seems to be bad. The following patch puts it early on the link command line like we do for -lasan or -ltsan. Bootstrapped/regtested on x86_64-linux and i686-linux, ok for tr