Re: [tsan] Small bugfix

2012-12-01 Thread Jakub Jelinek
On Sat, Dec 01, 2012 at 04:55:35PM +0400, Dmitry Vyukov wrote: > > No idea what do you mean by thread leak. What exactly is leaking? > > > Thread leak is joinable but not joined thread. > I have a pending todo to aggregate them by stack, so in this case it > will "3 threads leaked here". Perhaps

Re: [tsan] Small bugfix

2012-12-01 Thread Dmitry Vyukov
On Sat, Dec 1, 2012 at 4:23 PM, Jakub Jelinek wrote: >> > Hi! >> > >> > When I've tried to compile the attached testcase (I was trying to see >> > if tsan could discover the emutls.c data race), I got ICEs because >> > expr_ptr in certain cases wasn't is_gimple_val and thus was invalid to >> > pas

Re: [tsan] Small bugfix

2012-12-01 Thread Jakub Jelinek
On Sat, Dec 01, 2012 at 01:53:52PM +0400, Dmitry Vyukov wrote: > On Sat, Dec 1, 2012 at 4:04 AM, Jakub Jelinek wrote: > > Hi! > > > > When I've tried to compile the attached testcase (I was trying to see > > if tsan could discover the emutls.c data race), I got ICEs because > > expr_ptr in certain

Re: [tsan] Small bugfix

2012-12-01 Thread Dmitry Vyukov
On Sat, Dec 1, 2012 at 4:04 AM, Jakub Jelinek wrote: > Hi! > > When I've tried to compile the attached testcase (I was trying to see > if tsan could discover the emutls.c data race), I got ICEs because > expr_ptr in certain cases wasn't is_gimple_val and thus was invalid to > pass it directly to a