[Lldb-commits] [lldb] b783e5c - [lldb] Make the PlatformMacOSX unit test Apple specific

Jonas Devlieghere via lldb-commits lldb-commits at lists.llvm.org
Tue Mar 15 09:33:04 PDT 2022


Author: Jonas Devlieghere
Date: 2022-03-15T09:32:56-07:00
New Revision: b783e5c203c1959e04fc351a2e96f0700decc32c

URL: https://github.com/llvm/llvm-project/commit/b783e5c203c1959e04fc351a2e96f0700decc32c
DIFF: https://github.com/llvm/llvm-project/commit/b783e5c203c1959e04fc351a2e96f0700decc32c.diff

LOG: [lldb] Make the PlatformMacOSX unit test Apple specific

I thought that x86GetSupportedArchitectures would always return
x86_64-apple-macosx as a compatible architecture, regardless of the host
achitecture, but the Debian bot disagrees with that.

Added: 
    

Modified: 
    lldb/unittests/Platform/PlatformMacOSXTest.cpp

Removed: 
    


################################################################################
diff  --git a/lldb/unittests/Platform/PlatformMacOSXTest.cpp b/lldb/unittests/Platform/PlatformMacOSXTest.cpp
index e35489a47d87e..123fdbc39d672 100644
--- a/lldb/unittests/Platform/PlatformMacOSXTest.cpp
+++ b/lldb/unittests/Platform/PlatformMacOSXTest.cpp
@@ -21,6 +21,7 @@ class PlatformMacOSXTest : public ::testing::Test {
   SubsystemRAII<FileSystem, HostInfo, PlatformMacOSX> subsystems;
 };
 
+#ifdef __APPLE__
 static bool containsArch(const std::vector<ArchSpec> &archs,
                          const ArchSpec &arch) {
   return std::find_if(archs.begin(), archs.end(), [&](const ArchSpec &other) {
@@ -50,3 +51,4 @@ TEST_F(PlatformMacOSXTest, TestGetSupportedArchitectures) {
                             arm64_ios_arch));
 #endif
 }
+#endif


        


More information about the lldb-commits mailing list