[PATCH] Add -Wtsan.

2020-12-09 Thread Martin Liška
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/97868 * common.opt: Add new w

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

Re: [PATCH] Add -Wtsan.

2020-12-14 Thread Martin Liška
Sep 17 00:00:00 2001 From: Martin Liska Date: Mon, 7 Dec 2020 15:55:59 +0100 Subject: [PATCH] Add -Wtsan. gcc/ChangeLog: PR sanitizer/97868 * common.opt: Add new warning -Wtsan. * doc/invoke.texi: Likewise. * tsan.c (instrument_builtin_call): Warn users about unsupported std::atomic_thread_f

Re: [PATCH] Add -Wtsan.

2020-12-15 Thread Jeff Law via Gcc-patches
t; detail) I would suggest to follow that style. >> >> Martin > > Thank you Martin for the useful feedback. > > I've just tested the updated patch. > > Martin > > 0001-Add-Wtsan.patch > > From 7c982d06749fd3650825a5fb417e5e3487ecbcaf Mon

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