[libcxx-commits] [PATCH] D62454: SFINAE on pair/tuple assignment operators 2729

[Zoe Carver via Phabricator via libcxx-commits]

zoecarver added a comment.

@ldionne sorry about that. I don't know how I overlooked the majority of the issue. Anyway, I've made sure that all the overloads have a test now.



================
Comment at: libcxx/test/std/utilities/tuple/tuple.tuple/tuple.assign/move.pass.cpp:108
     {
-        using T = std::tuple<int, NonAssignable>;
-        static_assert(!std::is_move_assignable<T>::value, "");
+      // HERE (and below/above)
+      using T = std::tuple<int, NonAssignable>;
----------------
This is just to mark where we have existing test coverage for this issue (to make it easier to review). I'll remove these comments when I commit. 


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D62454/new/

https://reviews.llvm.org/D62454