diff options
author | Ville Voutilainen <ville.voutilainen@gmail.com> | 2016-12-01 18:23:21 +0200 |
---|---|---|
committer | Ville Voutilainen <ville@gcc.gnu.org> | 2016-12-01 18:23:21 +0200 |
commit | a2863bde755d39626ee25e3b7a8875e0d93d7217 (patch) | |
tree | a2bd7318c8afb8108d1a4d091102ab832cc467bb | |
parent | a9c21e2a4f44175eab8588d794a3ea3ce2fa8d0b (diff) | |
download | gcc-a2863bde755d39626ee25e3b7a8875e0d93d7217.zip gcc-a2863bde755d39626ee25e3b7a8875e0d93d7217.tar.gz gcc-a2863bde755d39626ee25e3b7a8875e0d93d7217.tar.bz2 |
Implement LWG 2766,
Swapping non-swappable types and LWG 2749,
swappable traits for variants.
* include/bits/move.h (swap(_Tp&, _Tp&)): Constrain
with __is_tuple_like.
* include/bits/stl_pair.h (swap(pair<_T1, _T2>&, pair<_T1, _T2>&)):
Add a deleted overload.
* include/bits/unique_ptr.h
(swap(unique_ptr<_Tp, _Dp>&, unique_ptr<_Tp, _Dp>&)): Likewise.
* include/std/array
(swap(array<_Tp, _Nm>&, array<_Tp, _Nm>&)): Likewise.
* include/std/optional
(swap(optional<_Tp>&, optional<_Tp>&)): Likewise.
* include/std/tuple (__is_tuple_like_impl, __is_tuple_like):
Move to type_traits.
(swap(tuple<_Elements...>&, tuple<_Elements...>&)): Add a deleted
overload.
* include/std/type_traits (__is_tuple_like_impl, __is_tuple_like):
New.
(swap(_Tp&, _Tp&)): Constrain with __is_tuple_like.
* include/std/utility (__is_tuple_like_impl): Move to type_traits.
* include/std/variant
(swap(variant<_Types...>&, variant<_Types...>&)):
Add a deleted overload.
* testsuite/20_util/optional/swap/2.cc: Add tests for disabled
swaps.
* testsuite/20_util/pair/swap_cxx17.cc: New.
* testsuite/20_util/tuple/swap_cxx17.cc: Likewise.
* testsuite/20_util/unique_ptr/specialized_algorithms/swap_cxx17.cc:
Likewise.
* testsuite/20_util/variant/compile.cc: Add tests for disabled
swaps.
* testsuite/23_containers/array/specialized_algorithms/swap_cxx17.cc:
New.
* testsuite/23_containers/array/tuple_interface/get_neg.cc: Adjust.
* testsuite/23_containers/array/tuple_interface/tuple_element_neg.cc:
Likewise.
From-SVN: r243120
18 files changed, 270 insertions, 24 deletions
diff --git a/libstdc++-v3/ChangeLog b/libstdc++-v3/ChangeLog index cc0b3ae..21404f16 100644 --- a/libstdc++-v3/ChangeLog +++ b/libstdc++-v3/ChangeLog @@ -1,5 +1,45 @@ 2016-12-01 Ville Voutilainen <ville.voutilainen@gmail.com> + Implement LWG 2766, + Swapping non-swappable types and LWG 2749, + swappable traits for variants. + * include/bits/move.h (swap(_Tp&, _Tp&)): Constrain + with __is_tuple_like. + * include/bits/stl_pair.h (swap(pair<_T1, _T2>&, pair<_T1, _T2>&)): + Add a deleted overload. + * include/bits/unique_ptr.h + (swap(unique_ptr<_Tp, _Dp>&, unique_ptr<_Tp, _Dp>&)): Likewise. + * include/std/array + (swap(array<_Tp, _Nm>&, array<_Tp, _Nm>&)): Likewise. + * include/std/optional + (swap(optional<_Tp>&, optional<_Tp>&)): Likewise. + * include/std/tuple (__is_tuple_like_impl, __is_tuple_like): + Move to type_traits. + (swap(tuple<_Elements...>&, tuple<_Elements...>&)): Add a deleted + overload. + * include/std/type_traits (__is_tuple_like_impl, __is_tuple_like): + New. + (swap(_Tp&, _Tp&)): Constrain with __is_tuple_like. + * include/std/utility (__is_tuple_like_impl): Move to type_traits. + * include/std/variant + (swap(variant<_Types...>&, variant<_Types...>&)): + Add a deleted overload. + * testsuite/20_util/optional/swap/2.cc: Add tests for disabled + swaps. + * testsuite/20_util/pair/swap_cxx17.cc: New. + * testsuite/20_util/tuple/swap_cxx17.cc: Likewise. + * testsuite/20_util/unique_ptr/specialized_algorithms/swap_cxx17.cc: + Likewise. + * testsuite/20_util/variant/compile.cc: Add tests for disabled + swaps. + * testsuite/23_containers/array/specialized_algorithms/swap_cxx17.cc: + New. + * testsuite/23_containers/array/tuple_interface/get_neg.cc: Adjust. + * testsuite/23_containers/array/tuple_interface/tuple_element_neg.cc: + Likewise. + +2016-12-01 Ville Voutilainen <ville.voutilainen@gmail.com> + The convertible_to traits need to use a variadic catch-all for the false-cases. * include/std/istream (__is_convertible_to_basic_istream): diff --git a/libstdc++-v3/include/bits/move.h b/libstdc++-v3/include/bits/move.h index d0aefe7..0bd11d6 100644 --- a/libstdc++-v3/include/bits/move.h +++ b/libstdc++-v3/include/bits/move.h @@ -181,7 +181,8 @@ _GLIBCXX_BEGIN_NAMESPACE_VERSION template<typename _Tp> inline #if __cplusplus >= 201103L - typename enable_if<__and_<is_move_constructible<_Tp>, + typename enable_if<__and_<__not_<__is_tuple_like<_Tp>>, + is_move_constructible<_Tp>, is_move_assignable<_Tp>>::value>::type swap(_Tp& __a, _Tp& __b) noexcept(__and_<is_nothrow_move_constructible<_Tp>, diff --git a/libstdc++-v3/include/bits/stl_pair.h b/libstdc++-v3/include/bits/stl_pair.h index ef52538..981dbeb 100644 --- a/libstdc++-v3/include/bits/stl_pair.h +++ b/libstdc++-v3/include/bits/stl_pair.h @@ -478,6 +478,14 @@ _GLIBCXX_BEGIN_NAMESPACE_VERSION swap(pair<_T1, _T2>& __x, pair<_T1, _T2>& __y) noexcept(noexcept(__x.swap(__y))) { __x.swap(__y); } + +#if __cplusplus > 201402L || !defined(__STRICT_ANSI__) // c++1z or gnu++11 + template<typename _T1, typename _T2> + inline + typename enable_if<!__and_<__is_swappable<_T1>, + __is_swappable<_T2>>::value>::type + swap(pair<_T1, _T2>&, pair<_T1, _T2>&) = delete; +#endif #endif // __cplusplus >= 201103L /** diff --git a/libstdc++-v3/include/bits/unique_ptr.h b/libstdc++-v3/include/bits/unique_ptr.h index f9ec60f..03f9bfc 100644 --- a/libstdc++-v3/include/bits/unique_ptr.h +++ b/libstdc++-v3/include/bits/unique_ptr.h @@ -650,6 +650,14 @@ _GLIBCXX_BEGIN_NAMESPACE_VERSION unique_ptr<_Tp, _Dp>& __y) noexcept { __x.swap(__y); } +#if __cplusplus > 201402L || !defined(__STRICT_ANSI__) // c++1z or gnu++11 + template<typename _Tp, typename _Dp> + inline + typename enable_if<!__is_swappable<_Dp>::value>::type + swap(unique_ptr<_Tp, _Dp>&, + unique_ptr<_Tp, _Dp>&) = delete; +#endif + template<typename _Tp, typename _Dp, typename _Up, typename _Ep> inline bool diff --git a/libstdc++-v3/include/std/array b/libstdc++-v3/include/std/array index 3ab0355..fa7bac6 100644 --- a/libstdc++-v3/include/std/array +++ b/libstdc++-v3/include/std/array @@ -288,6 +288,14 @@ _GLIBCXX_BEGIN_NAMESPACE_CONTAINER noexcept(noexcept(__one.swap(__two))) { __one.swap(__two); } +#if __cplusplus > 201402L || !defined(__STRICT_ANSI__) // c++1z or gnu++11 + template<typename _Tp, std::size_t _Nm> + inline + typename enable_if< + !_GLIBCXX_STD_C::__array_traits<_Tp, _Nm>::_Is_swappable::value>::type + swap(array<_Tp, _Nm>&, array<_Tp, _Nm>&) = delete; +#endif + template<std::size_t _Int, typename _Tp, std::size_t _Nm> constexpr _Tp& get(array<_Tp, _Nm>& __arr) noexcept diff --git a/libstdc++-v3/include/std/optional b/libstdc++-v3/include/std/optional index ea673cc..191d64b 100644 --- a/libstdc++-v3/include/std/optional +++ b/libstdc++-v3/include/std/optional @@ -930,6 +930,10 @@ _GLIBCXX_BEGIN_NAMESPACE_VERSION { __lhs.swap(__rhs); } template<typename _Tp> + inline enable_if_t<!(is_move_constructible_v<_Tp> && is_swappable_v<_Tp>)> + swap(optional<_Tp>&, optional<_Tp>&) = delete; + + template<typename _Tp> constexpr optional<decay_t<_Tp>> make_optional(_Tp&& __t) { return optional<decay_t<_Tp>> { std::forward<_Tp>(__t) }; } diff --git a/libstdc++-v3/include/std/tuple b/libstdc++-v3/include/std/tuple index 63cacd4..fb2fd17 100644 --- a/libstdc++-v3/include/std/tuple +++ b/libstdc++-v3/include/std/tuple @@ -1442,17 +1442,6 @@ _GLIBCXX_BEGIN_NAMESPACE_VERSION forward_as_tuple(_Elements&&... __args) noexcept { return tuple<_Elements&&...>(std::forward<_Elements>(__args)...); } - template<typename... _Tps> - struct __is_tuple_like_impl<tuple<_Tps...>> : true_type - { }; - - // Internal type trait that allows us to sfinae-protect tuple_cat. - template<typename _Tp> - struct __is_tuple_like - : public __is_tuple_like_impl<typename std::remove_cv - <typename std::remove_reference<_Tp>::type>::type>::type - { }; - template<size_t, typename, typename, size_t> struct __make_tuple_impl; @@ -1597,6 +1586,13 @@ _GLIBCXX_BEGIN_NAMESPACE_VERSION noexcept(noexcept(__x.swap(__y))) { __x.swap(__y); } +#if __cplusplus > 201402L || !defined(__STRICT_ANSI__) // c++1z or gnu++11 + template<typename... _Elements> + inline + typename enable_if<!__and_<__is_swappable<_Elements>...>::value>::type + swap(tuple<_Elements...>&, tuple<_Elements...>&) = delete; +#endif + // A class (and instance) which can be used in 'tie' when an element // of a tuple is not required struct _Swallow_assign diff --git a/libstdc++-v3/include/std/type_traits b/libstdc++-v3/include/std/type_traits index e5f2bba..f164f71 100644 --- a/libstdc++-v3/include/std/type_traits +++ b/libstdc++-v3/include/std/type_traits @@ -2593,9 +2593,28 @@ _GLIBCXX_BEGIN_NAMESPACE_VERSION template <typename _Tp> struct __is_nothrow_swappable; + template<typename... _Elements> + class tuple; + + template<typename> + struct __is_tuple_like_impl : false_type + { }; + + template<typename... _Tps> + struct __is_tuple_like_impl<tuple<_Tps...>> : true_type + { }; + + // Internal type trait that allows us to sfinae-protect tuple_cat. + template<typename _Tp> + struct __is_tuple_like + : public __is_tuple_like_impl<typename remove_cv< + typename remove_reference<_Tp>::type>::type>::type + { }; + template<typename _Tp> inline - typename enable_if<__and_<is_move_constructible<_Tp>, + typename enable_if<__and_<__not_<__is_tuple_like<_Tp>>, + is_move_constructible<_Tp>, is_move_assignable<_Tp>>::value>::type swap(_Tp&, _Tp&) noexcept(__and_<is_nothrow_move_constructible<_Tp>, diff --git a/libstdc++-v3/include/std/utility b/libstdc++-v3/include/std/utility index 3982156..8e02f0e 100644 --- a/libstdc++-v3/include/std/utility +++ b/libstdc++-v3/include/std/utility @@ -140,10 +140,6 @@ _GLIBCXX_BEGIN_NAMESPACE_VERSION using tuple_element_t = typename tuple_element<__i, _Tp>::type; #endif - template<typename> - struct __is_tuple_like_impl : false_type - { }; - // Various functions which give std::pair a tuple-like interface. /// Partial specialization for std::pair diff --git a/libstdc++-v3/include/std/variant b/libstdc++-v3/include/std/variant index 34ad3fd..89ca979 100644 --- a/libstdc++-v3/include/std/variant +++ b/libstdc++-v3/include/std/variant @@ -889,10 +889,17 @@ _GLIBCXX_BEGIN_NAMESPACE_VERSION { return false; } template<typename... _Types> - inline auto swap(variant<_Types...>& __lhs, variant<_Types...>& __rhs) - noexcept(noexcept(__lhs.swap(__rhs))) -> decltype(__lhs.swap(__rhs)) + inline enable_if_t<__and_<is_move_constructible<_Types>..., + is_swappable<_Types>...>::value> + swap(variant<_Types...>& __lhs, variant<_Types...>& __rhs) + noexcept(noexcept(__lhs.swap(__rhs))) { __lhs.swap(__rhs); } + template<typename... _Types> + inline enable_if_t<!__and_<is_move_constructible<_Types>..., + is_swappable<_Types>...>::value> + swap(variant<_Types...>&, variant<_Types...>&) = delete; + class bad_variant_access : public exception { public: diff --git a/libstdc++-v3/testsuite/20_util/optional/swap/2.cc b/libstdc++-v3/testsuite/20_util/optional/swap/2.cc index 5793488..cb9291a 100644 --- a/libstdc++-v3/testsuite/20_util/optional/swap/2.cc +++ b/libstdc++-v3/testsuite/20_util/optional/swap/2.cc @@ -33,11 +33,11 @@ void swap(B&, B&) noexcept(false); static_assert( std::is_swappable_v<std::optional<B>> ); static_assert( !std::is_nothrow_swappable_v<std::optional<B>> ); -// Not swappable, but optional<C> is swappable via the generic std::swap. +// Not swappable, and optional<C> not swappable via the generic std::swap. struct C { }; void swap(C&, C&) = delete; -static_assert( std::is_swappable_v<std::optional<C>> ); +static_assert( !std::is_swappable_v<std::optional<C>> ); // Not swappable, and optional<D> not swappable via the generic std::swap. struct D { D(D&&) = delete; }; diff --git a/libstdc++-v3/testsuite/20_util/pair/swap_cxx17.cc b/libstdc++-v3/testsuite/20_util/pair/swap_cxx17.cc new file mode 100644 index 0000000..6b09f42 --- /dev/null +++ b/libstdc++-v3/testsuite/20_util/pair/swap_cxx17.cc @@ -0,0 +1,35 @@ +// { dg-options "-std=gnu++17" } +// { dg-do compile } + +// Copyright (C) 2016 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> + +// Not swappable, and pair not swappable via the generic std::swap. +struct C { }; +void swap(C&, C&) = delete; + +static_assert( !std::is_swappable_v<std::pair<int, C>> ); +static_assert( !std::is_swappable_v<std::pair<C, int>> ); + +// Not swappable, and pair not swappable via the generic std::swap. +struct D { D(D&&) = delete; }; + +static_assert( !std::is_swappable_v<std::pair<int, D>> ); +static_assert( !std::is_swappable_v<std::pair<D, int>> ); diff --git a/libstdc++-v3/testsuite/20_util/tuple/swap_cxx17.cc b/libstdc++-v3/testsuite/20_util/tuple/swap_cxx17.cc new file mode 100644 index 0000000..d2a75ce --- /dev/null +++ b/libstdc++-v3/testsuite/20_util/tuple/swap_cxx17.cc @@ -0,0 +1,43 @@ +// { dg-options "-std=gnu++17" } +// { dg-do compile } + +// Copyright (C) 2016 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/>. + + +// NOTE: This makes use of the fact that we know how moveable +// is implemented on tuple. If the implementation changed +// this test may begin to fail. + +#include <tuple> + +// Not swappable, and tuple not swappable via the generic std::swap. +struct C { }; +void swap(C&, C&) = delete; + +static_assert( !std::is_swappable_v<std::tuple<int, C>> ); +static_assert( !std::is_swappable_v<std::tuple<C, int>> ); +static_assert( !std::is_swappable_v<std::tuple<int, int, C>> ); +static_assert( !std::is_swappable_v<std::tuple<C, int, int>> ); + +// Not swappable, and tuple not swappable via the generic std::swap. +struct D { D(D&&) = delete; }; + +static_assert( !std::is_swappable_v<std::tuple<int, D>> ); +static_assert( !std::is_swappable_v<std::tuple<D, int>> ); +static_assert( !std::is_swappable_v<std::tuple<int, int, D>> ); +static_assert( !std::is_swappable_v<std::tuple<D, int, int>> ); diff --git a/libstdc++-v3/testsuite/20_util/unique_ptr/specialized_algorithms/swap_cxx17.cc b/libstdc++-v3/testsuite/20_util/unique_ptr/specialized_algorithms/swap_cxx17.cc new file mode 100644 index 0000000..bf106ec --- /dev/null +++ b/libstdc++-v3/testsuite/20_util/unique_ptr/specialized_algorithms/swap_cxx17.cc @@ -0,0 +1,33 @@ +// { dg-options "-std=gnu++17" } +// { dg-do compile } + +// Copyright (C) 2016 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 <memory> + +// Not swappable, and unique_ptr not swappable via the generic std::swap. +struct C { }; +void swap(C&, C&) = delete; + +static_assert( !std::is_swappable_v<std::unique_ptr<int, C>> ); + +// Not swappable, and unique_ptr not swappable via the generic std::swap. +struct D { D(D&&) = delete; }; + +static_assert( !std::is_swappable_v<std::unique_ptr<int, D>> ); + diff --git a/libstdc++-v3/testsuite/20_util/variant/compile.cc b/libstdc++-v3/testsuite/20_util/variant/compile.cc index e3330be..8250a95 100644 --- a/libstdc++-v3/testsuite/20_util/variant/compile.cc +++ b/libstdc++-v3/testsuite/20_util/variant/compile.cc @@ -219,6 +219,21 @@ void test_relational() } } +// Not swappable, and variant<C> not swappable via the generic std::swap. +struct C { }; +void swap(C&, C&) = delete; + +static_assert( !std::is_swappable_v<variant<C>> ); +static_assert( !std::is_swappable_v<variant<int, C>> ); +static_assert( !std::is_swappable_v<variant<C, int>> ); + +// Not swappable, and variant<D> not swappable via the generic std::swap. +struct D { D(D&&) = delete; }; + +static_assert( !std::is_swappable_v<variant<D>> ); +static_assert( !std::is_swappable_v<variant<int, D>> ); +static_assert( !std::is_swappable_v<variant<D, int>> ); + void test_swap() { variant<int, string> a, b; diff --git a/libstdc++-v3/testsuite/23_containers/array/specialized_algorithms/swap_cxx17.cc b/libstdc++-v3/testsuite/23_containers/array/specialized_algorithms/swap_cxx17.cc new file mode 100644 index 0000000..2e93c4d --- /dev/null +++ b/libstdc++-v3/testsuite/23_containers/array/specialized_algorithms/swap_cxx17.cc @@ -0,0 +1,33 @@ +// { dg-options "-std=gnu++17" } +// { dg-do compile } + +// Copyright (C) 2016 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> + +// Not swappable, and pair not swappable via the generic std::swap. +struct C { }; +void swap(C&, C&) = delete; + +static_assert( !std::is_swappable_v<std::array<C, 42>> ); + +// Not swappable, and pair not swappable via the generic std::swap. +struct D { D(D&&) = delete; }; + +static_assert( !std::is_swappable_v<std::array<D, 42>> ); + diff --git a/libstdc++-v3/testsuite/23_containers/array/tuple_interface/get_neg.cc b/libstdc++-v3/testsuite/23_containers/array/tuple_interface/get_neg.cc index f99bbf6..568ec85 100644 --- a/libstdc++-v3/testsuite/23_containers/array/tuple_interface/get_neg.cc +++ b/libstdc++-v3/testsuite/23_containers/array/tuple_interface/get_neg.cc @@ -27,6 +27,6 @@ int n1 = std::get<1>(a); int n2 = std::get<1>(std::move(a)); int n3 = std::get<1>(ca); -// { dg-error "static assertion failed" "" { target *-*-* } 295 } -// { dg-error "static assertion failed" "" { target *-*-* } 304 } +// { dg-error "static assertion failed" "" { target *-*-* } 303 } // { dg-error "static assertion failed" "" { target *-*-* } 312 } +// { dg-error "static assertion failed" "" { target *-*-* } 320 } diff --git a/libstdc++-v3/testsuite/23_containers/array/tuple_interface/tuple_element_neg.cc b/libstdc++-v3/testsuite/23_containers/array/tuple_interface/tuple_element_neg.cc index 1941f3c..32cb10b 100644 --- a/libstdc++-v3/testsuite/23_containers/array/tuple_interface/tuple_element_neg.cc +++ b/libstdc++-v3/testsuite/23_containers/array/tuple_interface/tuple_element_neg.cc @@ -22,4 +22,4 @@ typedef std::tuple_element<1, std::array<int, 1>>::type type; -// { dg-error "static assertion failed" "" { target *-*-* } 343 } +// { dg-error "static assertion failed" "" { target *-*-* } 351 } |