Skip to content

Commit a56c621

Browse files
committed
[test] Disable TSan tests over remote-run
Ideally lit.cfg would conditionalize this based on the remote OS, but that's tricky. Meanwhile, let's not fail on old versions of Apple OSs that don't support TSan. rdar://problem/44802353
1 parent 84bb580 commit a56c621

File tree

5 files changed

+20
-0
lines changed

5 files changed

+20
-0
lines changed

test/SILGen/tsan_instrumentation.swift

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,10 @@
11
// RUN: %target-swift-emit-silgen -sanitize=thread %s | %FileCheck %s
22
// REQUIRES: tsan_runtime
33

4+
// FIXME: This should be covered by "tsan_runtime"; older versions of Apple OSs
5+
// don't support TSan.
6+
// UNSUPPORTED: remote_run
7+
48
func takesInout(_ p: inout Int) { }
59
func takesInout(_ p: inout MyStruct) { }
610

test/Sanitizers/tsan-norace-block-release.swift

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,10 @@
55
// REQUIRES: objc_interop
66
// REQUIRES: tsan_runtime
77

8+
// FIXME: This should be covered by "tsan_runtime"; older versions of Apple OSs
9+
// don't support TSan.
10+
// UNSUPPORTED: remote_run
11+
812
// Test that we do not report a race on block release operation.
913
import Foundation
1014

test/Sanitizers/tsan-norace-deinit-run-time.swift

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,10 @@
55
// REQUIRES: objc_interop
66
// REQUIRES: tsan_runtime
77

8+
// FIXME: This should be covered by "tsan_runtime"; older versions of Apple OSs
9+
// don't support TSan.
10+
// UNSUPPORTED: remote_run
11+
812
// Test that we do not report a race on deinit; the synchronization is guaranteed by runtime.
913
import Foundation
1014

test/Sanitizers/tsan-static-exclusivity.swift

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,10 @@
11
// RUN: %target-swift-frontend -enforce-exclusivity=checked -sanitize=thread -emit-sil -primary-file %s -o /dev/null -verify
22
// REQUIRES: tsan_runtime
33

4+
// FIXME: This should be covered by "tsan_runtime"; older versions of Apple OSs
5+
// don't support TSan.
6+
// UNSUPPORTED: remote_run
7+
48

59
struct OtherStruct {
610
mutating

test/Sanitizers/tsan.swift

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,10 @@
55
// REQUIRES: tsan_runtime
66
// UNSUPPORTED: OS=tvos
77

8+
// FIXME: This should be covered by "tsan_runtime"; older versions of Apple OSs
9+
// don't support TSan.
10+
// UNSUPPORTED: remote_run
11+
812
// https://bugs.swift.org/browse/SR-6622
913
// XFAIL: linux
1014

0 commit comments

Comments
 (0)