RPCSX/llvm
1
0
Fork 0
mirror of https://github.com/RPCSX/llvm.git synced 2024-11-24 20:29:53 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

[tsan] Don't instrument __llvm_gcov_global_state_pred or __llvm_gcda*

Browse Source 
r274801 did not go far enough to allow gcov+tsan to cooperate. With this
commit it's possible to run the following code without false positives:

  std::thread T1(fib), T2(fib);
  T1.join(); T2.join();

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@276015 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
Vedant Kumar 2016-07-19 20:16:08 +00:00
parent b2e69d912a
commit ea47fc5fc9
2 changed files with 16 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
lib/Transforms/Instrumentation/ThreadSanitizer.cpp
View File

@ -272,8 +272,9 @@ static bool shouldInstrumentReadWriteFromAddress(Value *Addr) {
return false;
}
// Check if the global is in a GCOV counter array.
if (GV->getName().startswith("__llvm_gcov_ctr"))
// Check if the global is private gcov data.
if (GV->getName().startswith("__llvm_gcov") ||
GV->getName().startswith("__llvm_gcda"))
return false;
}

13
test/Instrumentation/ThreadSanitizer/do-not-instrument-memory-access.ll
View File

@ -13,6 +13,8 @@ target triple = "x86_64-apple-macosx10.9"
@__llvm_gcov_ctr = internal global [1 x i64] zeroinitializer
@__llvm_gcov_ctr.1 = internal global [1 x i64] zeroinitializer
@__llvm_gcov_global_state_pred = internal global i32 0
@__llvm_gcda_foo = internal global i32 0
define i32 @test_gep() sanitize_thread {
entry:
@ -42,5 +44,16 @@ entry:
ret i32 undef
}
define void @test_load() sanitize_thread {
entry:
%0 = load i32, i32* @__llvm_gcov_global_state_pred
store i32 1, i32* @__llvm_gcov_global_state_pred
%1 = load i32, i32* @__llvm_gcda_foo
store i32 1, i32* @__llvm_gcda_foo
ret void
}
; CHECK-NOT: {{call void @__tsan_write}}
; CHECK: __tsan_init