From 189f88ca35cd038da52bfdb338e37a0c15fa69bb Mon Sep 17 00:00:00 2001 From: Eric Fiselier Date: Mon, 25 Jul 2016 01:45:07 +0000 Subject: Revert r276548 - Make pair/tuples assignment operators SFINAE properly. This is a breaking change. The SFINAE required is instantiated the second the class is instantiated, and this can cause hard SFINAE errors when applied to references to incomplete types. Ex. struct IncompleteType; extern IncompleteType it; std::tuple t(it); // SFINAE will blow up. llvm-svn: 276598 --- .../tuple/tuple.tuple/tuple.assign/move.pass.cpp | 69 ---------------------- 1 file changed, 69 deletions(-) (limited to 'libcxx/test/std/utilities/tuple/tuple.tuple/tuple.assign/move.pass.cpp') diff --git a/libcxx/test/std/utilities/tuple/tuple.tuple/tuple.assign/move.pass.cpp b/libcxx/test/std/utilities/tuple/tuple.tuple/tuple.assign/move.pass.cpp index 210f14be318..fc5e41ad569 100644 --- a/libcxx/test/std/utilities/tuple/tuple.tuple/tuple.assign/move.pass.cpp +++ b/libcxx/test/std/utilities/tuple/tuple.tuple/tuple.assign/move.pass.cpp @@ -21,33 +21,6 @@ #include "MoveOnly.h" -struct NonAssignable { - NonAssignable& operator=(NonAssignable const&) = delete; - NonAssignable& operator=(NonAssignable&&) = delete; -}; -struct CopyAssignable { - CopyAssignable& operator=(CopyAssignable const&) = default; - CopyAssignable& operator=(CopyAssignable&&) = delete; -}; -static_assert(std::is_copy_assignable::value, ""); -struct MoveAssignable { - MoveAssignable& operator=(MoveAssignable const&) = delete; - MoveAssignable& operator=(MoveAssignable&&) = default; -}; - - -struct CountAssign { - static int copied; - static int moved; - static void reset() { copied = moved = 0; } - CountAssign() = default; - CountAssign& operator=(CountAssign const&) { ++copied; return *this; } - CountAssign& operator=(CountAssign&&) { ++moved; return *this; } -}; -int CountAssign::copied = 0; -int CountAssign::moved = 0; - - int main() { { @@ -80,46 +53,4 @@ int main() assert(std::get<1>(t) == 1); assert(std::get<2>(t) == 2); } - { - // test reference assignment. - using T = std::tuple; - int x = 42; - int y = 100; - int x2 = -1; - int y2 = 500; - T t(x, std::move(y)); - T t2(x2, std::move(y2)); - t = std::move(t2); - assert(std::get<0>(t) == x2); - assert(&std::get<0>(t) == &x); - assert(std::get<1>(t) == y2); - assert(&std::get<1>(t) == &y); - } - { - // test that the implicitly generated move assignment operator - // is properly deleted - using T = std::tuple>; - static_assert(std::is_move_assignable::value, ""); - static_assert(!std::is_copy_assignable::value, ""); - - } - { - using T = std::tuple; - static_assert(!std::is_move_assignable::value, ""); - } - { - using T = std::tuple; - static_assert(std::is_move_assignable::value, ""); - } - { - // The move should decay to a copy. - CountAssign::reset(); - using T = std::tuple; - static_assert(std::is_move_assignable::value, ""); - T t1; - T t2; - t1 = std::move(t2); - assert(CountAssign::copied == 1); - assert(CountAssign::moved == 0); - } } -- cgit v1.2.3