Fix 01b02a73de78 to use correct macro spelling and fix unit tests.
authorAmy Huang <akhuang@google.com>
Fri, 14 Feb 2020 23:31:45 +0000 (15:31 -0800)
committerAmy Huang <akhuang@google.com>
Fri, 14 Feb 2020 23:58:36 +0000 (15:58 -0800)
llvm/lib/Support/Host.cpp
llvm/unittests/Support/Host.cpp

index 371271e..cafdc2f 100644 (file)
@@ -1326,7 +1326,7 @@ int computeHostNumPhysicalCores() {
   }
   return count;
 }
-#elif defined(_WIN32) && LLVM_THREADS_ENABLED
+#elif defined(_WIN32) && LLVM_ENABLE_THREADS != 0
 // Defined in llvm/lib/Support/Windows/Threading.inc
 int computeHostNumPhysicalCores();
 #else
index 6225234..e5c2816 100644 (file)
@@ -13,6 +13,7 @@
 #include "llvm/Support/FileSystem.h"
 #include "llvm/Support/Path.h"
 #include "llvm/Support/Program.h"
+#include "llvm/Support/Threading.h"
 
 #include "gtest/gtest.h"
 
@@ -37,7 +38,7 @@ protected:
     // Initially this is only testing detection of the number of
     // physical cores, which is currently only supported/tested for
     // x86_64 Linux and Darwin.
-    return Host.isOSWindows() ||
+    return (Host.isOSWindows() && llvm_is_multithreaded()) ||
            (Host.getArch() == Triple::x86_64 &&
             (Host.isOSDarwin() || Host.getOS() == Triple::Linux));
   }