[libcxx-commits] [libcxx] r353206 - [libc++] Use UNSUPPORTED instead of TEST_STD_VER #ifdef

Louis Dionne via libcxx-commits libcxx-commits at lists.llvm.org
Tue Feb 5 11:50:17 PST 2019


Author: ldionne
Date: Tue Feb  5 11:50:17 2019
New Revision: 353206

URL: http://llvm.org/viewvc/llvm-project?rev=353206&view=rev
Log:
[libc++] Use UNSUPPORTED instead of TEST_STD_VER #ifdef

When the whole test only works starting at some version of the Standard,
use UNSUPPORTED lit markup instead of #ifdef TEST_STD_VER. This provides
more visibility into the test suite.

Reviewed as https://reviews.llvm.org/D57704.
Thanks to Andrey Maksimov for the patch.

Modified:
    libcxx/trunk/test/std/input.output/iostream.format/quoted.manip/quoted.pass.cpp
    libcxx/trunk/test/std/utilities/allocator.adaptor/allocator.adaptor.types/allocator_pointers.pass.cpp
    libcxx/trunk/test/std/utilities/memory/default.allocator/allocator_pointers.pass.cpp
    libcxx/trunk/test/std/utilities/meta/meta.unary/meta.unary.cat/nullptr.pass.cpp
    libcxx/trunk/test/std/utilities/tuple/tuple.tuple/TupleFunction.pass.cpp

Modified: libcxx/trunk/test/std/input.output/iostream.format/quoted.manip/quoted.pass.cpp
URL: http://llvm.org/viewvc/llvm-project/libcxx/trunk/test/std/input.output/iostream.format/quoted.manip/quoted.pass.cpp?rev=353206&r1=353205&r2=353206&view=diff
==============================================================================
--- libcxx/trunk/test/std/input.output/iostream.format/quoted.manip/quoted.pass.cpp (original)
+++ libcxx/trunk/test/std/input.output/iostream.format/quoted.manip/quoted.pass.cpp Tue Feb  5 11:50:17 2019
@@ -6,6 +6,8 @@
 //
 //===----------------------------------------------------------------------===//
 
+// UNSUPPORTED: c++98, c++03, c++11
+
 // <iomanip>
 
 // quoted
@@ -15,10 +17,6 @@
 #include <string>
 #include <cassert>
 
-#include "test_macros.h"
-
-#if TEST_STD_VER > 11
-
 template <class CharT, class Traits>
 bool is_skipws ( const std::basic_istream<CharT, Traits>& is ) {
     return ( is.flags() & std::ios_base::skipws ) != 0;
@@ -173,12 +171,6 @@ int main(int, char**)
     assert ( unquote (  "" ) ==  "" ); // nothing there
     assert ( unquote ( L"" ) == L"" ); // nothing there
     test_padding ();
-    
-    return 0;
-}
 
-#else
-int main(int, char**) {
-  return 0;
+    return 0;
 }
-#endif

Modified: libcxx/trunk/test/std/utilities/allocator.adaptor/allocator.adaptor.types/allocator_pointers.pass.cpp
URL: http://llvm.org/viewvc/llvm-project/libcxx/trunk/test/std/utilities/allocator.adaptor/allocator.adaptor.types/allocator_pointers.pass.cpp?rev=353206&r1=353205&r2=353206&view=diff
==============================================================================
--- libcxx/trunk/test/std/utilities/allocator.adaptor/allocator.adaptor.types/allocator_pointers.pass.cpp (original)
+++ libcxx/trunk/test/std/utilities/allocator.adaptor/allocator.adaptor.types/allocator_pointers.pass.cpp Tue Feb  5 11:50:17 2019
@@ -6,13 +6,12 @@
 //
 //===----------------------------------------------------------------------===//
 
+// UNSUPPORTED: c++98, c++03
+
 #include <scoped_allocator>
 #include <memory>
 #include <cassert>
 
-#include "test_macros.h"
-
-#if TEST_STD_VER >= 11
 // #include <memory>
 //
 // template <class Alloc>
@@ -121,6 +120,3 @@ int main(int, char**)
 
   return 0;
 }
-#else
-int main(int, char**) { return 0; }
-#endif

Modified: libcxx/trunk/test/std/utilities/memory/default.allocator/allocator_pointers.pass.cpp
URL: http://llvm.org/viewvc/llvm-project/libcxx/trunk/test/std/utilities/memory/default.allocator/allocator_pointers.pass.cpp?rev=353206&r1=353205&r2=353206&view=diff
==============================================================================
--- libcxx/trunk/test/std/utilities/memory/default.allocator/allocator_pointers.pass.cpp (original)
+++ libcxx/trunk/test/std/utilities/memory/default.allocator/allocator_pointers.pass.cpp Tue Feb  5 11:50:17 2019
@@ -6,12 +6,11 @@
 //
 //===----------------------------------------------------------------------===//
 
+// UNSUPPORTED: c++98, c++03
+
 #include <memory>
 #include <cassert>
 
-#include "test_macros.h"
-
-#if TEST_STD_VER >= 11
 // #include <memory>
 //
 // template <class Alloc>
@@ -120,6 +119,3 @@ int main(int, char**)
 
   return 0;
 }
-#else
-int main(int, char**) { return 0; }
-#endif

Modified: libcxx/trunk/test/std/utilities/meta/meta.unary/meta.unary.cat/nullptr.pass.cpp
URL: http://llvm.org/viewvc/llvm-project/libcxx/trunk/test/std/utilities/meta/meta.unary/meta.unary.cat/nullptr.pass.cpp?rev=353206&r1=353205&r2=353206&view=diff
==============================================================================
--- libcxx/trunk/test/std/utilities/meta/meta.unary/meta.unary.cat/nullptr.pass.cpp (original)
+++ libcxx/trunk/test/std/utilities/meta/meta.unary/meta.unary.cat/nullptr.pass.cpp Tue Feb  5 11:50:17 2019
@@ -11,11 +11,11 @@
 // nullptr_t
 //  is_null_pointer
 
+// UNSUPPORTED: c++98, c++03, c++11
+
 #include <type_traits>
 #include <cstddef>        // for std::nullptr_t
-#include "test_macros.h"
 
-#if TEST_STD_VER > 11
 template <class T>
 void test_nullptr_imp()
 {
@@ -54,8 +54,3 @@ int main(int, char**)
     static_assert(!std::is_null_pointer<incomplete_type>::value, "");
     return 0;
 }
-#else
-int main(int, char**) {
-  return 0;
-}
-#endif

Modified: libcxx/trunk/test/std/utilities/tuple/tuple.tuple/TupleFunction.pass.cpp
URL: http://llvm.org/viewvc/llvm-project/libcxx/trunk/test/std/utilities/tuple/tuple.tuple/TupleFunction.pass.cpp?rev=353206&r1=353205&r2=353206&view=diff
==============================================================================
--- libcxx/trunk/test/std/utilities/tuple/tuple.tuple/TupleFunction.pass.cpp (original)
+++ libcxx/trunk/test/std/utilities/tuple/tuple.tuple/TupleFunction.pass.cpp Tue Feb  5 11:50:17 2019
@@ -6,11 +6,9 @@
 //
 //===----------------------------------------------------------------------===//
 
-// This is for bugs 18853 and 19118
-
-#include "test_macros.h"
+// UNSUPPORTED: c++98, c++03
 
-#if TEST_STD_VER >= 11
+// This is for bugs 18853 and 19118
 
 #include <tuple>
 #include <functional>
@@ -32,6 +30,3 @@ int main(int, char**)
 
   return 0;
 }
-#else
-int main(int, char**) { return 0; }
-#endif




More information about the libcxx-commits mailing list