[libcxx][test] Use TEST_HAS_BUILTIN in test code

... rather than using `__has_builtin` directly. This both (1) allows a compiler that doesn't speak `__has_builtin` to workaround with preprocessor magic, and (2) avoids diagnostics about things that look like function like macros after `#if` but are not.
llvm · Jan 20, 2022 · 67d483a · 67d483a
1 parent 60b6e73
commit 67d483a
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/...support/support.coroutines/coroutine.handle/coroutine.handle.noop/noop_coroutine.pass.cpp b/...support/support.coroutines/coroutine.handle/coroutine.handle.noop/noop_coroutine.pass.cpp
@@ -20,7 +20,7 @@
 
 #include "test_macros.h"
 
-#if __has_builtin(__builtin_coro_noop)
+#if TEST_HAS_BUILTIN(__builtin_coro_noop)
 
 namespace coro = std::experimental::coroutines_v1;
 
@@ -74,4 +74,4 @@ int main(int, char**)
 
 int main(int, char**) { return 0; }
 
-#endif //  __has_builtin(__builtin_coro_noop)
+#endif //  TEST_HAS_BUILTIN(__builtin_coro_noop)
diff --git a/libcxx/test/support/test_macros.h b/libcxx/test/support/test_macros.h
@@ -147,7 +147,7 @@
 
 #if defined(__cpp_lib_is_constant_evaluated) && __cpp_lib_is_constant_evaluated >= 201811L
 # define TEST_IS_CONSTANT_EVALUATED std::is_constant_evaluated()
-#elif __has_builtin(__builtin_is_constant_evaluated)
+#elif TEST_HAS_BUILTIN(__builtin_is_constant_evaluated)
 # define TEST_IS_CONSTANT_EVALUATED __builtin_is_constant_evaluated()
 #else
 # define TEST_IS_CONSTANT_EVALUATED false