From 3fc0d94ce57de2d0841e77c8fda7feef2923c4e0 Mon Sep 17 00:00:00 2001 From: "A. Jiang" Date: Sat, 26 Oct 2024 13:46:59 +0800 Subject: [libc++][test] Make macro detection more friendly to MSVC (#113633) MSVC STL's test suite is a bit nervous about replacing non-macro-defined identifiers with `0` (see also https://learn.microsoft.com/en-us/cpp/error-messages/compiler-warnings/compiler-warning-level-4-c4668?view=msvc-170). On MSVC (and MS-compatible mode of other compilers), `long double` has the same format (IEEE-754 binary64) as `double`, so it should be OK to define `TEST_LONG_DOUBLE_IS_DOUBLE` when `_MSC_VER` is defined. Such detection should be performed first. --- libcxx/test/support/test_macros.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libcxx/test/support/test_macros.h b/libcxx/test/support/test_macros.h index 5ef14e54dae2..1b6473b623c5 100644 --- a/libcxx/test/support/test_macros.h +++ b/libcxx/test/support/test_macros.h @@ -511,7 +511,7 @@ inline Tp const& DoNotOptimize(Tp const& value) { # define TEST_CONSTEXPR_OPERATOR_NEW #endif -#if __SIZEOF_LONG_DOUBLE__ == __SIZEOF_DOUBLE__ +#if defined(_MSC_VER) || __SIZEOF_LONG_DOUBLE__ == __SIZEOF_DOUBLE__ # define TEST_LONG_DOUBLE_IS_DOUBLE #endif -- cgit v1.2.3