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
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
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
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
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