aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJonathan Wakely <jwakely@redhat.com>2016-01-26 19:38:20 +0000
committerJonathan Wakely <redi@gcc.gnu.org>2016-01-26 19:38:20 +0000
commitf1d591e8a6b82935ec9767b3da52daa89d22f7a1 (patch)
tree377bdc5779146d2b593d7ebd63754ceb1700f7b7
parent2a4dfe66dac91a0f60c5864996df61e2a5ad625b (diff)
downloadgcc-f1d591e8a6b82935ec9767b3da52daa89d22f7a1.zip
gcc-f1d591e8a6b82935ec9767b3da52daa89d22f7a1.tar.gz
gcc-f1d591e8a6b82935ec9767b3da52daa89d22f7a1.tar.bz2
Fix assertions for move assignment of trivial types
PR libstdc++/69478 * include/bits/stl_algobase.h (__copy_move<_IsMove, true, random_access_iterator_tag>): Check is_move_assignable when moving. (__copy_move_backwards<_IsMove, true, random_access_iterator_tag>): Likewise. * testsuite/25_algorithms/copy/move_iterators/69478.cc: New. * testsuite/25_algorithms/copy_backward/move_iterators/69478.cc: New. * testsuite/25_algorithms/move/69478.cc: New. * testsuite/25_algorithms/move_backward/69478.cc: new. From-SVN: r232842
-rw-r--r--libstdc++-v3/ChangeLog12
-rw-r--r--libstdc++-v3/include/bits/stl_algobase.h12
-rw-r--r--libstdc++-v3/testsuite/25_algorithms/copy/move_iterators/69478.cc40
-rw-r--r--libstdc++-v3/testsuite/25_algorithms/copy_backward/move_iterators/69478.cc40
-rw-r--r--libstdc++-v3/testsuite/25_algorithms/move/69478.cc40
-rw-r--r--libstdc++-v3/testsuite/25_algorithms/move_backward/69478.cc40
6 files changed, 180 insertions, 4 deletions
diff --git a/libstdc++-v3/ChangeLog b/libstdc++-v3/ChangeLog
index b96afe6..4ac1e7f 100644
--- a/libstdc++-v3/ChangeLog
+++ b/libstdc++-v3/ChangeLog
@@ -1,3 +1,15 @@
+2016-01-26 Jonathan Wakely <jwakely@redhat.com>
+
+ PR libstdc++/69478
+ * include/bits/stl_algobase.h (__copy_move<_IsMove, true,
+ random_access_iterator_tag>): Check is_move_assignable when moving.
+ (__copy_move_backwards<_IsMove, true, random_access_iterator_tag>):
+ Likewise.
+ * testsuite/25_algorithms/copy/move_iterators/69478.cc: New.
+ * testsuite/25_algorithms/copy_backward/move_iterators/69478.cc: New.
+ * testsuite/25_algorithms/move/69478.cc: New.
+ * testsuite/25_algorithms/move_backward/69478.cc: new.
+
2016-01-26 Andris Pavenis <andris.pavenis@iki.fi>
* include/c_compatibility/stdlib.h: Include wide character related
diff --git a/libstdc++-v3/include/bits/stl_algobase.h b/libstdc++-v3/include/bits/stl_algobase.h
index 4a618be..d95ea51 100644
--- a/libstdc++-v3/include/bits/stl_algobase.h
+++ b/libstdc++-v3/include/bits/stl_algobase.h
@@ -357,9 +357,11 @@ _GLIBCXX_BEGIN_NAMESPACE_VERSION
__copy_m(const _Tp* __first, const _Tp* __last, _Tp* __result)
{
#if __cplusplus >= 201103L
+ using __assignable = conditional<_IsMove,
+ is_move_assignable<_Tp>,
+ is_copy_assignable<_Tp>>;
// trivial types can have deleted assignment
- static_assert( is_copy_assignable<_Tp>::value,
- "type is not assignable" );
+ static_assert( __assignable::type::value, "type is not assignable" );
#endif
const ptrdiff_t _Num = __last - __first;
if (_Num)
@@ -557,9 +559,11 @@ _GLIBCXX_BEGIN_NAMESPACE_VERSION
__copy_move_b(const _Tp* __first, const _Tp* __last, _Tp* __result)
{
#if __cplusplus >= 201103L
+ using __assignable = conditional<_IsMove,
+ is_move_assignable<_Tp>,
+ is_copy_assignable<_Tp>>;
// trivial types can have deleted assignment
- static_assert( is_copy_assignable<_Tp>::value,
- "type is not assignable" );
+ static_assert( __assignable::type::value, "type is not assignable" );
#endif
const ptrdiff_t _Num = __last - __first;
if (_Num)
diff --git a/libstdc++-v3/testsuite/25_algorithms/copy/move_iterators/69478.cc b/libstdc++-v3/testsuite/25_algorithms/copy/move_iterators/69478.cc
new file mode 100644
index 0000000..707b273
--- /dev/null
+++ b/libstdc++-v3/testsuite/25_algorithms/copy/move_iterators/69478.cc
@@ -0,0 +1,40 @@
+// { dg-options "-std=gnu++11" }
+// { 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/>.
+
+// PR libstdc++/69478
+
+#include <algorithm>
+#include <iterator>
+
+void
+test01()
+{
+ // A move-only type that is also a trivial class.
+ struct trivial_rvalstruct
+ {
+ trivial_rvalstruct() = default;
+ trivial_rvalstruct(trivial_rvalstruct&&) = default;
+ trivial_rvalstruct& operator=(trivial_rvalstruct&&) = default;
+ };
+ static_assert(std::is_trivial<trivial_rvalstruct>::value, "");
+
+ trivial_rvalstruct a[1], b[1];
+ copy(std::make_move_iterator(a), std::make_move_iterator(a + 1), b);
+}
diff --git a/libstdc++-v3/testsuite/25_algorithms/copy_backward/move_iterators/69478.cc b/libstdc++-v3/testsuite/25_algorithms/copy_backward/move_iterators/69478.cc
new file mode 100644
index 0000000..e00d146
--- /dev/null
+++ b/libstdc++-v3/testsuite/25_algorithms/copy_backward/move_iterators/69478.cc
@@ -0,0 +1,40 @@
+// { dg-options "-std=gnu++11" }
+// { 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/>.
+
+// PR libstdc++/69478
+
+#include <algorithm>
+#include <iterator>
+
+void
+test01()
+{
+ // A move-only type that is also a trivial class.
+ struct trivial_rvalstruct
+ {
+ trivial_rvalstruct() = default;
+ trivial_rvalstruct(trivial_rvalstruct&&) = default;
+ trivial_rvalstruct& operator=(trivial_rvalstruct&&) = default;
+ };
+ static_assert(std::is_trivial<trivial_rvalstruct>::value, "");
+
+ trivial_rvalstruct a[1], b[1];
+ copy_backward(std::make_move_iterator(a), std::make_move_iterator(a+1), b);
+}
diff --git a/libstdc++-v3/testsuite/25_algorithms/move/69478.cc b/libstdc++-v3/testsuite/25_algorithms/move/69478.cc
new file mode 100644
index 0000000..791e0bf
--- /dev/null
+++ b/libstdc++-v3/testsuite/25_algorithms/move/69478.cc
@@ -0,0 +1,40 @@
+// { dg-options "-std=gnu++11" }
+// { 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/>.
+
+// PR libstdc++/69478
+
+#include <algorithm>
+#include <iterator>
+
+void
+test01()
+{
+ // A move-only type that is also a trivial class.
+ struct trivial_rvalstruct
+ {
+ trivial_rvalstruct() = default;
+ trivial_rvalstruct(trivial_rvalstruct&&) = default;
+ trivial_rvalstruct& operator=(trivial_rvalstruct&&) = default;
+ };
+ static_assert(std::is_trivial<trivial_rvalstruct>::value, "");
+
+ trivial_rvalstruct a[1], b[1];
+ std::move(a, a + 1, b);
+}
diff --git a/libstdc++-v3/testsuite/25_algorithms/move_backward/69478.cc b/libstdc++-v3/testsuite/25_algorithms/move_backward/69478.cc
new file mode 100644
index 0000000..8ca198f
--- /dev/null
+++ b/libstdc++-v3/testsuite/25_algorithms/move_backward/69478.cc
@@ -0,0 +1,40 @@
+// { dg-options "-std=gnu++11" }
+// { 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/>.
+
+// PR libstdc++/69478
+
+#include <algorithm>
+#include <iterator>
+
+void
+test01()
+{
+ // A move-only type that is also a trivial class.
+ struct trivial_rvalstruct
+ {
+ trivial_rvalstruct() = default;
+ trivial_rvalstruct(trivial_rvalstruct&&) = default;
+ trivial_rvalstruct& operator=(trivial_rvalstruct&&) = default;
+ };
+ static_assert(std::is_trivial<trivial_rvalstruct>::value, "");
+
+ trivial_rvalstruct a[1], b[1];
+ std::move_backward(a, a + 1, b);
+}