[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