[libcxx-commits] [libcxx] a5faf3c - [libc++] Re-enable tests for C11 math macros in <float.h> and <cfloat>

Louis Dionne via libcxx-commits libcxx-commits at lists.llvm.org
Wed Jul 29 12:13:05 PDT 2020


Author: Louis Dionne
Date: 2020-07-29T15:12:54-04:00
New Revision: a5faf3c849298d1106e74068f3ecffa13e1684a6

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

LOG: [libc++] Re-enable tests for C11 math macros in <float.h> and <cfloat>

Fixes http://llvm.org/PR38572.

Added: 
    

Modified: 
    libcxx/test/std/depr/depr.c.headers/float_h.pass.cpp
    libcxx/test/std/language.support/support.limits/c.limits/cfloat.pass.cpp
    libcxx/test/support/test_macros.h

Removed: 
    


################################################################################
diff  --git a/libcxx/test/std/depr/depr.c.headers/float_h.pass.cpp b/libcxx/test/std/depr/depr.c.headers/float_h.pass.cpp
index 1069a679263f..2b7edcadc217 100644
--- a/libcxx/test/std/depr/depr.c.headers/float_h.pass.cpp
+++ b/libcxx/test/std/depr/depr.c.headers/float_h.pass.cpp
@@ -24,7 +24,7 @@
 #error FLT_RADIX not defined
 #endif
 
-#if TEST_STD_VER > 14 && 0
+#if TEST_STD_VER > 14
 #ifndef FLT_HAS_SUBNORM
 #error FLT_HAS_SUBNORM not defined
 #endif
@@ -54,7 +54,7 @@
 #error DECIMAL_DIG not defined
 #endif
 
-#if TEST_STD_VER > 14 && 0
+#if TEST_STD_VER > 14
 #ifndef FLT_DECIMAL_DIG
 #error FLT_DECIMAL_DIG not defined
 #endif
@@ -164,7 +164,7 @@
 #error LDBL_MIN not defined
 #endif
 
-#if TEST_STD_VER > 14 && 0
+#if TEST_STD_VER > 14
 #ifndef FLT_TRUE_MIN
 #error FLT_TRUE_MIN not defined
 #endif

diff  --git a/libcxx/test/std/language.support/support.limits/c.limits/cfloat.pass.cpp b/libcxx/test/std/language.support/support.limits/c.limits/cfloat.pass.cpp
index 6ddd8c60834b..99fe6a37243c 100644
--- a/libcxx/test/std/language.support/support.limits/c.limits/cfloat.pass.cpp
+++ b/libcxx/test/std/language.support/support.limits/c.limits/cfloat.pass.cpp
@@ -24,7 +24,7 @@
 #error FLT_RADIX not defined
 #endif
 
-#if TEST_STD_VER > 14 && 0
+#if TEST_STD_VER > 14
 #ifndef FLT_HAS_SUBNORM
 #error FLT_HAS_SUBNORM not defined
 #endif
@@ -54,7 +54,7 @@
 #error DECIMAL_DIG not defined
 #endif
 
-#if TEST_STD_VER > 14 && 0
+#if TEST_STD_VER > 14
 #ifndef FLT_DECIMAL_DIG
 #error FLT_DECIMAL_DIG not defined
 #endif
@@ -164,7 +164,7 @@
 #error LDBL_MIN not defined
 #endif
 
-#if TEST_STD_VER > 14 && 0
+#if TEST_STD_VER > 14
 #ifndef FLT_TRUE_MIN
 #error FLT_TRUE_MIN not defined
 #endif

diff  --git a/libcxx/test/support/test_macros.h b/libcxx/test/support/test_macros.h
index 6fe1c56a5588..00873607fb11 100644
--- a/libcxx/test/support/test_macros.h
+++ b/libcxx/test/support/test_macros.h
@@ -162,7 +162,6 @@
 #endif
 
 // Sniff out to see if the underlying C library has C11 features
-// Note that at this time (July 2018), MacOS X and iOS do NOT.
 // This is cribbed from __config; but lives here as well because we can't assume libc++
 #if __ISO_C_VISIBLE >= 2011 || TEST_STD_VER >= 11
 #  if defined(__FreeBSD__)


        


More information about the libcxx-commits mailing list