From e21b7273c9097c5c30ef89563c5c38ca09a1165e Mon Sep 17 00:00:00 2001 From: Christopher Dykes Date: Thu, 21 Jul 2016 16:09:26 -0700 Subject: [PATCH] Disable a test of ThreadLocal across forks for Windows Summary: We don't have `fork()` to begin with, so disable the test and the includes it required if we're compiling on Windows. Reviewed By: yfeldblum Differential Revision: D3600600 fbshipit-source-id: 5399705753b456139105f4ac757190e1ba1765f2 --- folly/test/ThreadLocalTest.cpp | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/folly/test/ThreadLocalTest.cpp b/folly/test/ThreadLocalTest.cpp index 7a3dab1d..e836ef37 100644 --- a/folly/test/ThreadLocalTest.cpp +++ b/folly/test/ThreadLocalTest.cpp @@ -16,9 +16,12 @@ #include +#ifndef _WIN32 #include -#include #include +#endif + +#include #include #include @@ -543,6 +546,7 @@ TEST(ThreadLocal, Fork) { } #endif +#ifndef _WIN32 struct HoldsOneTag2 {}; TEST(ThreadLocal, Fork2) { @@ -571,6 +575,7 @@ TEST(ThreadLocal, Fork2) { EXPECT_TRUE(false) << "fork failed"; } } +#endif // Elide this test when using any sanitizer. Otherwise, the dlopen'ed code // would end up running without e.g., ASAN-initialized data structures and -- 2.34.1