[PATCH] D14907: [tsan] Fix weakly imported functions on OS X

Kuba Brecka via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 23 02:49:15 PST 2015


kubabrecka updated this revision to Diff 40908.
kubabrecka added a comment.

The previous patch failed several unit tests, because `OnReport` needs to be exported as a C++ symbol (in `__tsan` namespace).  Changing the patch to have both `SANITIZER_WEAK_DEFAULT_IMPL` (for C symbols) and `SANITIZER_WEAK_CXX_DEFAULT_IMPL` (for C++ symbols).


http://reviews.llvm.org/D14907

Files:
  lib/sanitizer_common/sanitizer_internal_defs.h
  lib/tsan/rtl/tsan_defs.h
  lib/tsan/rtl/tsan_flags.cc
  lib/tsan/rtl/tsan_mman.cc
  lib/tsan/rtl/tsan_rtl.cc
  lib/tsan/rtl/tsan_rtl_report.cc
  lib/tsan/rtl/tsan_suppressions.cc
  lib/tsan/rtl/tsan_symbolize.cc

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D14907.40908.patch
Type: text/x-patch
Size: 4592 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20151123/c2e10e66/attachment.bin>


More information about the llvm-commits mailing list