[compiler-rt] [tsan][test] Move the linux specific TSan test to the linux subdirectory (PR #86896)
via llvm-commits
llvm-commits at lists.llvm.org
Wed Mar 27 17:55:30 PDT 2024
=?utf-8?q?Nazım_Can_Altınova?= <canaltinova at gmail.com>
Message-ID:
In-Reply-To: <llvm.org/llvm/llvm-project/pull/86896 at github.com>
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-compiler-rt-sanitizer
Author: Nazım Can Altınova (canova)
<details>
<summary>Changes</summary>
I introduced this test in #<!-- -->86537 but it should only be run on Linux since it's tests Linux syscalls specifically.
This PR moves it to `compiler-rt/test/tsan/Linux/` subdirectory, and removes some unneeded debugging comments in the meantime.
---
Full diff: https://github.com/llvm/llvm-project/pull/86896.diff
1 Files Affected:
- (renamed) compiler-rt/test/tsan/Linux/signal_in_futex_wait.cpp (-3)
``````````diff
diff --git a/compiler-rt/test/tsan/signal_in_futex_wait.cpp b/compiler-rt/test/tsan/Linux/signal_in_futex_wait.cpp
similarity index 94%
rename from compiler-rt/test/tsan/signal_in_futex_wait.cpp
rename to compiler-rt/test/tsan/Linux/signal_in_futex_wait.cpp
index cf31e5467486ad..485d87d5ee846c 100644
--- a/compiler-rt/test/tsan/signal_in_futex_wait.cpp
+++ b/compiler-rt/test/tsan/Linux/signal_in_futex_wait.cpp
@@ -57,16 +57,13 @@ class Mutex {
Mutex mutex;
void *Thread(void *x) {
- // fprintf(stderr, "canova here thread 0\n");
// Waiting for the futex.
mutex.lock();
- // fprintf(stderr, "canova here thread 1\n");
// Finished waiting.
return nullptr;
}
static void SigprofHandler(int signal, siginfo_t *info, void *context) {
- // fprintf(stderr, "canova here sigprof handler\n");
// Unlock the futex.
mutex.unlock();
}
``````````
</details>
https://github.com/llvm/llvm-project/pull/86896
More information about the llvm-commits
mailing list