Skip to content

Conversation

@DavidSpickett
Copy link
Collaborator

As is done for other procfile tests.

@DavidSpickett DavidSpickett added the skip-precommit-approval PR for CI feedback, not intended for review label Sep 18, 2025
@llvmbot llvmbot added the lldb label Sep 18, 2025
@llvmbot
Copy link
Member

llvmbot commented Sep 18, 2025

@llvm/pr-subscribers-lldb

Author: David Spickett (DavidSpickett)

Changes

As is done for other procfile tests.


Full diff: https://github.com/llvm/llvm-project/pull/159559.diff

1 Files Affected:

  • (modified) lldb/unittests/Host/posix/SupportTest.cpp (+2-2)
diff --git a/lldb/unittests/Host/posix/SupportTest.cpp b/lldb/unittests/Host/posix/SupportTest.cpp
index 5393ad83ad8ac..1b11f0caa0981 100644
--- a/lldb/unittests/Host/posix/SupportTest.cpp
+++ b/lldb/unittests/Host/posix/SupportTest.cpp
@@ -26,7 +26,7 @@ TEST(Support, getProcFile_Pid) {
 }
 #endif // #ifndef __APPLE__
 
-#if defined(_AIX) || defined(__linux__)
+#if (defined(_AIX) || defined(__linux__)) && defined(LLVM_ENABLE_THREADING)
 TEST(Support, getProcFile_Tid) {
   auto BufferOrError = getProcFile(getpid(), llvm::get_threadid(),
 #ifdef _AIX
@@ -38,4 +38,4 @@ TEST(Support, getProcFile_Tid) {
   ASSERT_TRUE(BufferOrError);
   ASSERT_TRUE(*BufferOrError);
 }
-#endif // #if defined(_AIX) || defined(__linux__)
+#endif // #if (defined(_AIX) || defined(__linux__)) && defined(LLVM_ENABLE_THREADING)

@github-actions
Copy link

github-actions bot commented Sep 18, 2025

✅ With the latest revision this PR passed the C/C++ code formatter.

@DavidSpickett DavidSpickett merged commit 1fc9b34 into llvm:main Sep 18, 2025
6 of 8 checks passed
@DavidSpickett DavidSpickett deleted the lldb-test-thread branch September 18, 2025 12:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lldb skip-precommit-approval PR for CI feedback, not intended for review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants