re PR libstdc++/66693 ([C++17] std::tuple_size fails with const std::array)
authorVille Voutilainen <ville.voutilainen@gmail.com>
Mon, 21 Dec 2015 11:22:16 +0000 (13:22 +0200)
committerVille Voutilainen <ville@gcc.gnu.org>
Mon, 21 Dec 2015 11:22:16 +0000 (13:22 +0200)
PR libstdc++/66693.

* include/std/tuple (tuple_element, tuple_size, tuple_element_t,
__tuple_element_t): Move to...
* include/std/utility: ...here.
* testsuite/20_util/pair/astuple/astuple.cc: Adjust.
* testsuite/20_util/pair/astuple/astuple_cpp14.cc: New.
* testsuite/20_util/tuple/tuple_element.cc: Adjust.
* testsuite/20_util/tuple/tuple_element_t.cc: Likewise.
* testsuite/20_util/tuple/tuple_size.cc: Likewise.
* testsuite/23_containers/array/tuple_interface/tuple_element.cc:
Likewise.
* testsuite/23_containers/array/tuple_interface/tuple_element_cpp14.cc:
New.
* testsuite/23_containers/array/tuple_interface/tuple_size.cc: Adjust.

From-SVN: r231875

libstdc++-v3/ChangeLog
libstdc++-v3/include/std/tuple
libstdc++-v3/include/std/utility
libstdc++-v3/testsuite/20_util/pair/astuple/astuple.cc
libstdc++-v3/testsuite/20_util/pair/astuple/astuple_cpp14.cc [new file with mode: 0644]
libstdc++-v3/testsuite/20_util/tuple/tuple_element.cc
libstdc++-v3/testsuite/20_util/tuple/tuple_element_t.cc
libstdc++-v3/testsuite/20_util/tuple/tuple_size.cc
libstdc++-v3/testsuite/23_containers/array/tuple_interface/tuple_element.cc
libstdc++-v3/testsuite/23_containers/array/tuple_interface/tuple_element_cpp14.cc [new file with mode: 0644]
libstdc++-v3/testsuite/23_containers/array/tuple_interface/tuple_size.cc

index c4f9f3c2e0a37f55e6395896f499256ad548c4fd..1a79ff8212e5622404a9738c70b3bbba57971b87 100644 (file)
@@ -1,3 +1,20 @@
+2015-12-21  Ville Voutilainen  <ville.voutilainen@gmail.com>
+
+       PR libstdc++/66693
+       * include/std/tuple (tuple_element, tuple_size, tuple_element_t,
+       __tuple_element_t): Move to...
+       * include/std/utility: ...here.
+       * testsuite/20_util/pair/astuple/astuple.cc: Adjust.
+       * testsuite/20_util/pair/astuple/astuple_cpp14.cc: New.
+       * testsuite/20_util/tuple/tuple_element.cc: Adjust.
+       * testsuite/20_util/tuple/tuple_element_t.cc: Likewise.
+       * testsuite/20_util/tuple/tuple_size.cc: Likewise.
+       * testsuite/23_containers/array/tuple_interface/tuple_element.cc:
+       Likewise.
+       * testsuite/23_containers/array/tuple_interface/tuple_element_cpp14.cc:
+       New.
+       * testsuite/23_containers/array/tuple_interface/tuple_size.cc: Adjust.
+
 2015-12-18  Ville Voutilainen  <ville.voutilainen@gmail.com>
 
        Fix a regression introduced by the fix of libstdc++/68276.
index e6c32b348be12fa2207b302e29deb93fe86d21ac..78f58b4ce434c5f2c13746a4d7bfdddc6e78529d 100644 (file)
@@ -1197,10 +1197,6 @@ _GLIBCXX_BEGIN_NAMESPACE_VERSION
     };
 
 
-  /// Gives the type of the ith element of a given tuple type.
-  template<std::size_t __i, typename _Tp>
-    struct tuple_element;
-
   /**
    * Recursive case for tuple_element: strip off the first element in
    * the tuple and retrieve the (i-1)th element of the remaining tuple.
@@ -1218,53 +1214,6 @@ _GLIBCXX_BEGIN_NAMESPACE_VERSION
       typedef _Head type;
     };
 
-  // Duplicate of C++14's tuple_element_t for internal use in C++11 mode
-  template<std::size_t __i, typename _Tp>
-    using __tuple_element_t = typename tuple_element<__i, _Tp>::type;
-
-  template<std::size_t __i, typename _Tp>
-    struct tuple_element<__i, const _Tp>
-    {
-      typedef typename add_const<__tuple_element_t<__i, _Tp>>::type type;
-    };
-
-  template<std::size_t __i, typename _Tp>
-    struct tuple_element<__i, volatile _Tp>
-    {
-      typedef typename add_volatile<__tuple_element_t<__i, _Tp>>::type type;
-    };
-
-  template<std::size_t __i, typename _Tp>
-    struct tuple_element<__i, const volatile _Tp>
-    {
-      typedef typename add_cv<__tuple_element_t<__i, _Tp>>::type type;
-    };
-
-#if __cplusplus > 201103L
-#define __cpp_lib_tuple_element_t 201402
-
-  template<std::size_t __i, typename _Tp>
-    using tuple_element_t = typename tuple_element<__i, _Tp>::type;
-#endif
-
-  /// Finds the size of a given tuple type.
-  template<typename _Tp>
-    struct tuple_size;
-
-  // _GLIBCXX_RESOLVE_LIB_DEFECTS
-  // 2313. tuple_size should always derive from integral_constant<size_t, N>
-  template<typename _Tp>
-    struct tuple_size<const _Tp>
-    : integral_constant<size_t, tuple_size<_Tp>::value> { };
-
-  template<typename _Tp>
-    struct tuple_size<volatile _Tp>
-    : integral_constant<size_t, tuple_size<_Tp>::value> { };
-
-  template<typename _Tp>
-    struct tuple_size<const volatile _Tp>
-    : integral_constant<size_t, tuple_size<_Tp>::value> { };
-
   /// class tuple_size
   template<typename... _Elements>
     struct tuple_size<tuple<_Elements...>>
index 985bcb202156802792a0f5a1f5ac1403f316c64e..2115d73cac6524501c8dab1d3b98c72ce2f40003 100644 (file)
@@ -79,11 +79,56 @@ namespace std _GLIBCXX_VISIBILITY(default)
 {
 _GLIBCXX_BEGIN_NAMESPACE_VERSION
 
-  template<class _Tp>
-    class tuple_size;
+  /// Finds the size of a given tuple type.
+  template<typename _Tp>
+    struct tuple_size;
+
+  // _GLIBCXX_RESOLVE_LIB_DEFECTS
+  // 2313. tuple_size should always derive from integral_constant<size_t, N>
+  template<typename _Tp>
+    struct tuple_size<const _Tp>
+    : integral_constant<size_t, tuple_size<_Tp>::value> { };
+
+  template<typename _Tp>
+    struct tuple_size<volatile _Tp>
+    : integral_constant<size_t, tuple_size<_Tp>::value> { };
+
+  template<typename _Tp>
+    struct tuple_size<const volatile _Tp>
+    : integral_constant<size_t, tuple_size<_Tp>::value> { };
+
+  /// Gives the type of the ith element of a given tuple type.
+  template<std::size_t __i, typename _Tp>
+    struct tuple_element;
+
+  // Duplicate of C++14's tuple_element_t for internal use in C++11 mode
+  template<std::size_t __i, typename _Tp>
+    using __tuple_element_t = typename tuple_element<__i, _Tp>::type;
+
+  template<std::size_t __i, typename _Tp>
+    struct tuple_element<__i, const _Tp>
+    {
+      typedef typename add_const<__tuple_element_t<__i, _Tp>>::type type;
+    };
+
+  template<std::size_t __i, typename _Tp>
+    struct tuple_element<__i, volatile _Tp>
+    {
+      typedef typename add_volatile<__tuple_element_t<__i, _Tp>>::type type;
+    };
 
-  template<std::size_t _Int, class _Tp>
-    class tuple_element;
+  template<std::size_t __i, typename _Tp>
+    struct tuple_element<__i, const volatile _Tp>
+    {
+      typedef typename add_cv<__tuple_element_t<__i, _Tp>>::type type;
+    };
+
+#if __cplusplus > 201103L
+#define __cpp_lib_tuple_element_t 201402
+
+  template<std::size_t __i, typename _Tp>
+    using tuple_element_t = typename tuple_element<__i, _Tp>::type;
+#endif
 
   template<typename>
     struct __is_tuple_like_impl : false_type
index 11e3fce2411d3a594ccd4a4a413d53a4f5654a98..15ab87210eff5c661997640b7e07f773a50a3230 100644 (file)
 typedef std::pair<int, long> test_type;
 
 static_assert( std::tuple_size<test_type>::value == 2, "size is 2" );
+static_assert( std::tuple_size<const test_type>::value == 2, "size is 2" );
+static_assert( std::tuple_size<volatile test_type>::value == 2, "size is 2" );
+static_assert( std::tuple_size<const volatile test_type>::value == 2,
+               "size is 2" );
 
 template<std::size_t N, typename T>
   using Tuple_elt = typename std::tuple_element<N, T>::type;
@@ -35,3 +39,27 @@ static_assert( is_same<Tuple_elt<0, test_type>, test_type::first_type>::value,
 
 static_assert( is_same<Tuple_elt<1, test_type>, test_type::second_type>::value,
                "second type is long" );
+
+static_assert( is_same<Tuple_elt<0, const test_type>,
+               const test_type::first_type>::value,
+               "first type is const int" );
+
+static_assert( is_same<Tuple_elt<1, const test_type>,
+               const test_type::second_type>::value,
+               "second type is const long" );
+
+static_assert( is_same<Tuple_elt<0, volatile test_type>,
+               volatile test_type::first_type>::value,
+               "first type is volatile int" );
+
+static_assert( is_same<Tuple_elt<1, volatile test_type>,
+               volatile test_type::second_type>::value,
+               "second type is volatile long" );
+
+static_assert( is_same<Tuple_elt<0, const volatile test_type>,
+               const volatile test_type::first_type>::value,
+               "first type is const volatile int" );
+
+static_assert( is_same<Tuple_elt<1, const volatile test_type>,
+               const volatile test_type::second_type>::value,
+               "second type is const volatile long" );
diff --git a/libstdc++-v3/testsuite/20_util/pair/astuple/astuple_cpp14.cc b/libstdc++-v3/testsuite/20_util/pair/astuple/astuple_cpp14.cc
new file mode 100644 (file)
index 0000000..9a0a3e5
--- /dev/null
@@ -0,0 +1,59 @@
+// { dg-do compile }
+// { dg-options "-std=gnu++14" }
+
+// Copyright (C) 2015 Free Software Foundation, Inc.
+//
+// This file is part of the GNU ISO C++ Library.  This library is free
+// software; you can redistribute it and/or modify it under the
+// terms of the GNU General Public License as published by the
+// Free Software Foundation; either version 3, or (at your option)
+// any later version.
+//
+// This library is distributed in the hope that it will be useful,
+// but WITHOUT ANY WARRANTY; without even the implied warranty of
+// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+// GNU General Public License for more details.
+//
+// You should have received a copy of the GNU General Public License along
+// with this library; see the file COPYING3.  If not see
+// <http://www.gnu.org/licenses/>.
+
+#include <utility>
+#include <type_traits>
+
+typedef std::pair<int, long> test_type;
+
+template<std::size_t N, typename T>
+  using Tuple_elt = std::tuple_element_t<N, T>;
+
+using std::is_same;
+
+static_assert( is_same<Tuple_elt<0, test_type>, test_type::first_type>::value,
+               "first type is int" );
+
+static_assert( is_same<Tuple_elt<1, test_type>, test_type::second_type>::value,
+               "second type is long" );
+
+static_assert( is_same<Tuple_elt<0, const test_type>,
+               const test_type::first_type>::value,
+               "first type is const int" );
+
+static_assert( is_same<Tuple_elt<1, const test_type>,
+               const test_type::second_type>::value,
+               "second type is const long" );
+
+static_assert( is_same<Tuple_elt<0, volatile test_type>,
+               volatile test_type::first_type>::value,
+               "first type is volatile int" );
+
+static_assert( is_same<Tuple_elt<1, volatile test_type>,
+               volatile test_type::second_type>::value,
+               "second type is volatile long" );
+
+static_assert( is_same<Tuple_elt<0, const volatile test_type>,
+               const volatile test_type::first_type>::value,
+               "first type is const volatile int" );
+
+static_assert( is_same<Tuple_elt<1, const volatile test_type>,
+               const volatile test_type::second_type>::value,
+               "second type is const volatile long" );
index 78231cb1a11a99b1097ed4dff0ab5eeefe2e0103..9e6224ced7f4e43762da4c09868ddb1678ceaf9a 100644 (file)
@@ -35,4 +35,10 @@ main()
   foo q1;
   tuple_element<0,tuple<foo,void,int> >::type q2(q1);
   tuple_element<2,tuple<void,int,foo> >::type q3(q1);
+  tuple_element<0,const tuple<foo,void,int> >::type q4(q1);
+  tuple_element<2,const tuple<void,int,foo> >::type q5(q1);
+  tuple_element<0,volatile tuple<foo,void,int> >::type q6(q1);
+  tuple_element<2,volatile tuple<void,int,foo> >::type q7(q1);
+  tuple_element<0,const volatile tuple<foo,void,int> >::type q8(q1);
+  tuple_element<2,const volatile tuple<void,int,foo> >::type q9(q1);
 }
index 481666b3805ab8d1088765bda4ad0009e1511dd2..079e7f9879c460867912c2de5cdd39640d5d5f9f 100644 (file)
@@ -33,4 +33,10 @@ main()
   foo q1;
   tuple_element_t<0,tuple<foo,void,int> > q2(q1);
   tuple_element_t<2,tuple<void,int,foo> > q3(q1);
+  tuple_element_t<0,const tuple<foo,void,int> > q4(q1);
+  tuple_element_t<2,const tuple<void,int,foo> > q5(q1);
+  tuple_element_t<0,volatile tuple<foo,void,int> > q6(q1);
+  tuple_element_t<2,volatile tuple<void,int,foo> > q7(q1);
+  tuple_element_t<0,const volatile tuple<foo,void,int> > q8(q1);
+  tuple_element_t<2,const volatile tuple<void,int,foo> > q9(q1);
 }
index 98b2477e585ad48041190fe5d8958bc08dfe9738..d6f6ec5c5032e4a981674cd34aab4296825b417c 100644 (file)
@@ -1,3 +1,4 @@
+// { dg-do compile }
 // { dg-options "-std=gnu++11" }
 
 // Copyright (C) 2007-2015 Free Software Foundation, Inc.
@@ -29,10 +30,28 @@ main()
 {
   bool test __attribute__((unused)) = true;
 
-  VERIFY(tuple_size<tuple<> >::value == 0);
-  VERIFY(tuple_size<tuple<int> >::value == 1);
-  VERIFY(tuple_size<tuple<void> >::value == 1);
+  static_assert(tuple_size<tuple<>>::value == 0, "");
+  static_assert(tuple_size<tuple<int>>::value == 1, "");
+  static_assert(tuple_size<tuple<void>>::value == 1, "");
   typedef tuple<int,const int&,void> test_tuple1;
-  VERIFY(tuple_size<test_tuple1>::value == 3);
-  VERIFY(tuple_size<tuple<tuple<void> > >::value == 1);
+  static_assert(tuple_size<test_tuple1>::value == 3, "");
+  static_assert(tuple_size<tuple<tuple<void>>>::value == 1, "");
+
+  static_assert(tuple_size<const tuple<>>::value == 0, "");
+  static_assert(tuple_size<const tuple<int>>::value == 1, "");
+  static_assert(tuple_size<const tuple<void>>::value == 1, "");
+  static_assert(tuple_size<const test_tuple1>::value == 3, "");
+  static_assert(tuple_size<const tuple<tuple<void>>>::value == 1, "");
+
+  static_assert(tuple_size<volatile tuple<>>::value == 0, "");
+  static_assert(tuple_size<volatile tuple<int>>::value == 1, "");
+  static_assert(tuple_size<volatile tuple<void>>::value == 1, "");
+  static_assert(tuple_size<volatile test_tuple1>::value == 3, "");
+  static_assert(tuple_size<volatile tuple<tuple<void>>>::value == 1, "");
+
+  static_assert(tuple_size<const volatile tuple<>>::value == 0, "");
+  static_assert(tuple_size<const volatile tuple<int>>::value == 1, "");
+  static_assert(tuple_size<const volatile tuple<void>>::value == 1, "");
+  static_assert(tuple_size<const volatile test_tuple1>::value == 3, "");
+  static_assert(tuple_size<const volatile tuple<tuple<void>>>::value == 1,"");
 }
index dda293d93ec8b6109ae52be0d201ad0053e21f62..09ba7a27b540b800086091bc9cc1dc090c7795fa 100644 (file)
@@ -1,3 +1,4 @@
+// { dg-do compile }
 // { dg-options "-std=gnu++11" }
 //
 // Copyright (C) 2011-2015 Free Software Foundation, Inc.
@@ -29,9 +30,31 @@ test01()
 
   const size_t len = 3;
   typedef array<int, len> array_type;
-  VERIFY( (is_same<tuple_element<0, array_type>::type, int>::value == true) );
-  VERIFY( (is_same<tuple_element<1, array_type>::type, int>::value == true) );
-  VERIFY( (is_same<tuple_element<2, array_type>::type, int>::value == true) );
+
+  static_assert(is_same<tuple_element<0, array_type>::type, int>::value, "" );
+  static_assert(is_same<tuple_element<1, array_type>::type, int>::value, "" );
+  static_assert(is_same<tuple_element<2, array_type>::type, int>::value, "");
+
+  static_assert(is_same<tuple_element<0, const array_type>::type,
+                const int>::value, "");
+  static_assert(is_same<tuple_element<1, const array_type>::type,
+                const int>::value, "");
+  static_assert(is_same<tuple_element<2, const array_type>::type,
+                const int>::value, "");
+
+  static_assert(is_same<tuple_element<0, volatile array_type>::type,
+                volatile int>::value, "");
+  static_assert(is_same<tuple_element<1, volatile array_type>::type,
+                volatile int>::value, "");
+  static_assert( (is_same<tuple_element<2, volatile array_type>::type,
+           volatile int>::value == true) );
+
+  static_assert(is_same<tuple_element<0, const volatile array_type>::type,
+                const volatile int>::value, "");
+  static_assert(is_same<tuple_element<1, const volatile array_type>::type,
+                const volatile int>::value, "");
+  static_assert(is_same<tuple_element<2, const volatile array_type>::type,
+                const volatile int>::value, "");
 }
 
 int main()
diff --git a/libstdc++-v3/testsuite/23_containers/array/tuple_interface/tuple_element_cpp14.cc b/libstdc++-v3/testsuite/23_containers/array/tuple_interface/tuple_element_cpp14.cc
new file mode 100644 (file)
index 0000000..b885ea3
--- /dev/null
@@ -0,0 +1,64 @@
+// { dg-do compile }
+// { dg-options "-std=gnu++14" }
+//
+// Copyright (C) 2015 Free Software Foundation, Inc.
+//
+// This file is part of the GNU ISO C++ Library.  This library is free
+// software; you can redistribute it and/or modify it under the
+// terms of the GNU General Public License as published by the
+// Free Software Foundation; either version 3, or (at your option)
+// any later version.
+//
+// This library is distributed in the hope that it will be useful,
+// but WITHOUT ANY WARRANTY; without even the implied warranty of
+// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+// GNU General Public License for more details.
+//
+// You should have received a copy of the GNU General Public License along
+// with this library; see the file COPYING3.  If not see
+// <http://www.gnu.org/licenses/>.
+
+#include <array>
+#include <type_traits>
+#include <testsuite_hooks.h>
+
+void
+test01()
+{
+  bool test __attribute__((unused)) = true;
+  using namespace std;
+
+  const size_t len = 3;
+  typedef array<int, len> array_type;
+
+  static_assert(is_same<tuple_element_t<0, array_type>, int>::value, "");
+  static_assert(is_same<tuple_element_t<1, array_type>, int>::value, "");
+  static_assert(is_same<tuple_element_t<2, array_type>, int>::value, "");
+
+  static_assert(is_same<tuple_element_t<0, const array_type>,
+                const int>::value, "");
+  static_assert(is_same<tuple_element_t<1, const array_type>,
+                const int>::value, "");
+  static_assert(is_same<tuple_element_t<2, const array_type>,
+                const int>::value, "");
+
+  static_assert(is_same<tuple_element_t<0, volatile array_type>,
+                volatile int>::value, "");
+  static_assert(is_same<tuple_element_t<1, volatile array_type>,
+                volatile int>::value, "");
+  static_assert(is_same<tuple_element_t<2, volatile array_type>,
+                volatile int>::value, "");
+
+  static_assert(is_same<tuple_element_t<0, const volatile array_type>,
+                const volatile int>::value, "");
+  static_assert(is_same<tuple_element_t<1, const volatile array_type>,
+                const volatile int>::value, "");
+  static_assert(is_same<tuple_element_t<2, const volatile array_type>,
+                const volatile int>::value, "");
+}
+
+int main()
+{
+  test01();
+  return 0;
+}
index 59326857cc8206d145d82850be8a2e18b312caa0..a821fdeb55957176250c3cda6a4c07698872f06f 100644 (file)
@@ -1,3 +1,4 @@
+// { dg-do compile }
 // { dg-options "-std=gnu++11" }
 //
 // Copyright (C) 2011-2015 Free Software Foundation, Inc.
@@ -29,13 +30,19 @@ test01()
   {
     const size_t len = 5;
     typedef array<int, len> array_type;
-    VERIFY( tuple_size<array_type>::value == 5 );
+    static_assert(tuple_size<array_type>::value == 5, "");
+    static_assert(tuple_size<const array_type>::value == 5, "");
+    static_assert(tuple_size<volatile array_type>::value == 5, "");
+    static_assert(tuple_size<const volatile array_type>::value == 5, "");
   }
 
   {
     const size_t len = 0;
     typedef array<float, len> array_type;
-    VERIFY( tuple_size<array_type>::value == 0 );
+    static_assert(tuple_size<array_type>::value == 0, "");
+    static_assert(tuple_size<const array_type>::value == 0, "");
+    static_assert(tuple_size<volatile array_type>::value == 0, "");
+    static_assert(tuple_size<const volatile array_type>::value == 0, "");
   }
 }