diff options
author | Ivan Maidanski <ivmai@mail.ru> | 2022-04-02 15:06:05 +0300 |
---|---|---|
committer | Ivan Maidanski <ivmai@mail.ru> | 2022-04-02 15:06:05 +0300 |
commit | 3bb4322fb2a030decac4b555f6c4215951b5ae95 (patch) | |
tree | cb35d0474591b840d27a6daa7ada820683794d91 /pthread_stop_world.c | |
parent | ba0616b09bd3354ea9f27ec1ffc1480adbab78b8 (diff) | |
download | bdwgc-3bb4322fb2a030decac4b555f6c4215951b5ae95.tar.gz |
Revert "Workaround GCC TSan false positive in GC_lookup_thread_async"
This reverts commit deba6eb5b05beb9c3e6cb4652491300c633e194c.
There is a simpler solution covering all uses of
attribute((no_sanitize_thread)) - add attribute((noinline)) to
GC_ATTR_NO_SANITIZE_THREAD definition.
Diffstat (limited to 'pthread_stop_world.c')
-rw-r--r-- | pthread_stop_world.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/pthread_stop_world.c b/pthread_stop_world.c index cbb3b2b8..75ca3d8e 100644 --- a/pthread_stop_world.c +++ b/pthread_stop_world.c @@ -289,7 +289,7 @@ STATIC void GC_suspend_handler_inner(ptr_t dummy, void *context); #ifdef THREAD_SANITIZER /* The implementation of the function is the same as that of */ /* GC_lookup_thread except for the attribute added here. */ - GC_ATTR_NOINLINE GC_ATTR_NO_SANITIZE_THREAD + GC_ATTR_NO_SANITIZE_THREAD static GC_thread GC_lookup_thread_async(pthread_t id) { GC_thread p = GC_threads[THREAD_TABLE_INDEX(id)]; |