EricWF added inline comments.

================
Comment at: 
test/std/utilities/variant/variant.variant/variant.assign/copy.pass.cpp:389
+template<class...Ts>
+constexpr bool triviality_test =
+  std::is_trivially_copy_assignable<std::variant<Ts...>>::value ==
----------------
`triviality_test` should also compare to an explicitly specified expected 
result, along with checking for consistency between variant and its input types.


================
Comment at: 
test/std/utilities/variant/variant.variant/variant.ctor/copy.pass.cpp:168
+void test_triviality_extension() {
+#if defined(_MSVC_STL_VER)
+  static_assert(triviality_test<int>, "");
----------------
These tests pass with libc++ too, so please add `|| defined(_LIBCPP_VERSION)`


================
Comment at: 
test/std/utilities/variant/variant.variant/variant.ctor/move.pass.cpp:206
+void test_triviality_extension() {
+#if defined(_MSVC_STL_VER)
+  static_assert(triviality_test<int>, "");
----------------
These tests pass with libc++ too, so please add `|| defined(_LIBCPP_VERSION)`


https://reviews.llvm.org/D32671



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to