Skip to content

Commit 69f8740

Browse files
committed
[OpenMP][Archer][Tests] NFC: fix spurious test failure
The test disables suppression and therefore sometimes triggers a know false positive in the openmp runtime. The test should only verify that the env var is handles as expected.
1 parent e426ae2 commit 69f8740

File tree

2 files changed

+1
-3
lines changed

2 files changed

+1
-3
lines changed

openmp/tools/archer/tests/lit.cfg

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -111,7 +111,7 @@ config.substitutions.append(("%clang-archer", config.test_c_compiler))
111111
config.substitutions.append(("%openmp_flags", config.test_openmp_flags))
112112
config.substitutions.append(("%archer_flags", config.archer_flags))
113113
config.substitutions.append(("%flags", config.test_flags))
114-
config.substitutions.append(("%nosuppression", "env TSAN_OPTIONS='ignore_noninstrumented_modules=0'"))
114+
config.substitutions.append(("%nosuppression", "env TSAN_OPTIONS='ignore_noninstrumented_modules=0:exitcode=0'"))
115115
config.substitutions.append(("%suppression", "env TSAN_OPTIONS='ignore_noninstrumented_modules=0:ignore_noninstrumented_modules=1'"))
116116
config.substitutions.append(("%deflake", os.path.join(os.path.dirname(__file__), "deflake.bash")))
117117

openmp/tools/archer/tests/parallel/parallel-nosuppression.c

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,5 @@ int main(int argc, char *argv[]) {
3434
return error;
3535
}
3636

37-
// CHECK-NOT: ThreadSanitizer: data race
38-
// CHECK-NOT: ThreadSanitizer: reported
3937
// CHECK: Warning: please export TSAN_OPTIONS
4038
// CHECK: DONE

0 commit comments

Comments
 (0)