resend: Re: [PATCH] strace: Fix crash caused over-optimization

2017-04-19 Thread Daniel Santos
he inconvenience. You can add yourself to the cygwin.com "global allow list" by sending email *from*the*blocked*email*address* to: Forwarded Message -------- Subject: Re: [PATCH] strace: Fix crash caused over-optimization Date: Wed, 19 Apr 2017 10:57:02 -0500 Fr

Re: [PATCH] strace: Fix crash caused over-optimization

2017-04-19 Thread Daniel Santos
On 04/18/2017 05:04 AM, Corinna Vinschen wrote: On Apr 17 03:39, Daniel Santos wrote: I actually did try that, although I had guessed it wouldn't (and shouldn't) work. I believe that the reason is that rather the accesses are volatile or not, gcc can see nothing else using it and memset can be

Re: [PATCH] strace: Fix crash caused over-optimization

2017-04-18 Thread Corinna Vinschen
On Apr 17 03:39, Daniel Santos wrote: > On 04/16/2017 05:21 AM, Jon Turney wrote: > > On 15/04/2017 23:27, Daniel Santos wrote: > > > Recent versions of gcc are optimizing away the TLS buffer allocated in > > > main, so we need to tell gcc that it's really used. > > > --- > > > winsup/utils/strace

Re: [PATCH] strace: Fix crash caused over-optimization

2017-04-17 Thread Daniel Santos
On 04/16/2017 05:21 AM, Jon Turney wrote: On 15/04/2017 23:27, Daniel Santos wrote: Recent versions of gcc are optimizing away the TLS buffer allocated in main, so we need to tell gcc that it's really used. --- winsup/utils/strace.cc | 2 ++ 1 file changed, 2 insertions(+) diff --git a/winsup/

Re: [PATCH] strace: Fix crash caused over-optimization

2017-04-16 Thread Jon Turney
On 15/04/2017 23:27, Daniel Santos wrote: Recent versions of gcc are optimizing away the TLS buffer allocated in main, so we need to tell gcc that it's really used. --- winsup/utils/strace.cc | 2 ++ 1 file changed, 2 insertions(+) diff --git a/winsup/utils/strace.cc b/winsup/utils/strace.cc in