[compiler-rt] 16a395b - Add SANITIZER_CDECL to __tsan_check_no_mutexes_held (#71471)

via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 6 17:29:26 PST 2023


Author: Charlie Barto
Date: 2023-11-06T17:29:22-08:00
New Revision: 16a395b74d35c564f6f36ba4a167950a323badd9

URL: https://github.com/llvm/llvm-project/commit/16a395b74d35c564f6f36ba4a167950a323badd9
DIFF: https://github.com/llvm/llvm-project/commit/16a395b74d35c564f6f36ba4a167950a323badd9.diff

LOG: Add SANITIZER_CDECL to __tsan_check_no_mutexes_held (#71471)

in https://github.com/llvm/llvm-project/pull/69625 @strega-nil added
cdecl to a huge number of sanitizer interface declarations. It looks
like she was racing against @kennyyu adding a tsan interface function. I
noticed this when merging in the latest changes from llvm/main and
corrected it.

Co-authored-by: Charlie Barto <Charles.Barto at microsoft.com>

Added: 
    

Modified: 
    compiler-rt/include/sanitizer/tsan_interface.h

Removed: 
    


################################################################################
diff  --git a/compiler-rt/include/sanitizer/tsan_interface.h b/compiler-rt/include/sanitizer/tsan_interface.h
index 01c08fe41e4da95..e11a4175cd8ed9f 100644
--- a/compiler-rt/include/sanitizer/tsan_interface.h
+++ b/compiler-rt/include/sanitizer/tsan_interface.h
@@ -129,7 +129,7 @@ void SANITIZER_CDECL __tsan_mutex_post_divert(void *addr, unsigned flags);
 
 // Check that the current thread does not hold any mutexes,
 // report a bug report otherwise.
-void __tsan_check_no_mutexes_held();
+void SANITIZER_CDECL __tsan_check_no_mutexes_held();
 
 // External race detection API.
 // Can be used by non-instrumented libraries to detect when their objects are


        


More information about the llvm-commits mailing list