Re: [PATCH] Add -Wtsan.

2020-12-16 Thread Martin Liška
On 12/16/20 1:19 AM, Jeff Law wrote: OK.   Please consider a more through description in invoke.texi though.   What unsupported feature are we warning about and what are the consequences of not supporting that feature. Done that and installed the patch. Thank you for the review, Martin

Re: [PATCH] Add -Wtsan.

2020-12-15 Thread Jeff Law via Gcc-patches
On 12/14/20 3:09 AM, Martin Liška wrote: > On 12/10/20 4:50 PM, Martin Sebor wrote: >> Most similar warnings mention the sanitizer option rather than >> referring to the tool by name.  E.g., >> >>    "transactional memory is not supported with  >> %<-fsanitize=address%>"

Re: [PATCH] Add -Wtsan.

2020-12-14 Thread Martin Liška
On 12/10/20 4:50 PM, Martin Sebor wrote: Most similar warnings mention the sanitizer option rather than referring to the tool by name.  E.g.,   "transactional memory is not supported with %<-fsanitize=address%>" or   "%<-fsanitize=leak%> is incompatible with %<-fsanitize=thread%>" For the sa

Re: [PATCH] Add -Wtsan.

2020-12-10 Thread Martin Sebor via Gcc-patches
On 12/9/20 2:24 AM, Martin Liška wrote: Hello. The newly added warning is about warning a user that std::atomic_thread_fence is not supported by TSAN. Patch can bootstrap on x86_64-linux-gnu and survives regression tests. Ready to be installed? Thanks, Martin gcc/ChangeLog: PR sanitizer