Fix PR30979 - tuple<move_only> is constructible from move_only const&
git-svn-id: https://llvm.org/svn/llvm-project/libcxx/trunk@286774 91177308-0d34-0410-b5e6-96231b3b80d8
diff --git a/include/tuple b/include/tuple
index 7e1c942..9f6661f 100644
--- a/include/tuple
+++ b/include/tuple
@@ -632,7 +632,7 @@
<
_CheckArgsConstructor<
_Dummy
- >::template __enable_implicit<_Tp...>(),
+ >::template __enable_implicit<_Tp const&...>(),
bool
>::type = false
>
@@ -650,7 +650,7 @@
<
_CheckArgsConstructor<
_Dummy
- >::template __enable_explicit<_Tp...>(),
+ >::template __enable_explicit<_Tp const&...>(),
bool
>::type = false
>
@@ -668,7 +668,7 @@
<
_CheckArgsConstructor<
_Dummy
- >::template __enable_implicit<_Tp...>(),
+ >::template __enable_implicit<_Tp const&...>(),
bool
>::type = false
>
@@ -687,7 +687,7 @@
<
_CheckArgsConstructor<
_Dummy
- >::template __enable_explicit<_Tp...>(),
+ >::template __enable_explicit<_Tp const&...>(),
bool
>::type = false
>
diff --git a/test/std/utilities/tuple/tuple.tuple/tuple.cnstr/move.pass.cpp b/test/std/utilities/tuple/tuple.tuple/tuple.cnstr/move.pass.cpp
index 1bd7d6d..0c93673 100644
--- a/test/std/utilities/tuple/tuple.tuple/tuple.cnstr/move.pass.cpp
+++ b/test/std/utilities/tuple/tuple.tuple/tuple.cnstr/move.pass.cpp
@@ -35,6 +35,52 @@
}
};
+// move_only type which triggers the empty base optimization
+struct move_only_ebo {
+ move_only_ebo() = default;
+ move_only_ebo(move_only_ebo&&) = default;
+};
+
+// a move_only type which does not trigger the empty base optimization
+struct move_only_large final {
+ move_only_large() : value(42) {}
+ move_only_large(move_only_large&&) = default;
+ int value;
+};
+
+template <class Elem>
+void test_sfinae() {
+ using Tup = std::tuple<Elem>;
+ using Alloc = std::allocator<void>;
+ using Tag = std::allocator_arg_t;
+ // special members
+ {
+ static_assert(std::is_default_constructible<Tup>::value, "");
+ static_assert(std::is_move_constructible<Tup>::value, "");
+ static_assert(!std::is_copy_constructible<Tup>::value, "");
+ static_assert(!std::is_constructible<Tup, Tup&>::value, "");
+ }
+ // args constructors
+ {
+ static_assert(std::is_constructible<Tup, Elem&&>::value, "");
+ static_assert(!std::is_constructible<Tup, Elem const&>::value, "");
+ static_assert(!std::is_constructible<Tup, Elem&>::value, "");
+ }
+ // uses-allocator special member constructors
+ {
+ static_assert(std::is_constructible<Tup, Tag, Alloc>::value, "");
+ static_assert(std::is_constructible<Tup, Tag, Alloc, Tup&&>::value, "");
+ static_assert(!std::is_constructible<Tup, Tag, Alloc, Tup const&>::value, "");
+ static_assert(!std::is_constructible<Tup, Tag, Alloc, Tup &>::value, "");
+ }
+ // uses-allocator args constructors
+ {
+ static_assert(std::is_constructible<Tup, Tag, Alloc, Elem&&>::value, "");
+ static_assert(!std::is_constructible<Tup, Tag, Alloc, Elem const&>::value, "");
+ static_assert(!std::is_constructible<Tup, Tag, Alloc, Elem &>::value, "");
+ }
+}
+
int main()
{
{
@@ -72,4 +118,8 @@
d_t d((ConstructsWithTupleLeaf()));
d_t d2(static_cast<d_t &&>(d));
}
+ {
+ test_sfinae<move_only_ebo>();
+ test_sfinae<move_only_large>();
+ }
}