libstdc++: Fix tests that fail in C++20 mode
authorJonathan Wakely <jwakely@redhat.com>
Mon, 20 Apr 2020 21:06:32 +0000 (22:06 +0100)
committerJonathan Wakely <jwakely@redhat.com>
Mon, 20 Apr 2020 21:06:32 +0000 (22:06 +0100)
* testsuite/20_util/is_constructible/51185.cc: Make test class a
non-aggregate so that the test verifies the same thing in all -std
modes.
* testsuite/20_util/is_constructible/value-2.cc: Adjust expected
results for some types when paren-init for aggregates is supported.

libstdc++-v3/ChangeLog
libstdc++-v3/testsuite/20_util/is_constructible/51185.cc
libstdc++-v3/testsuite/20_util/is_constructible/value-2.cc

index 9f4c9d327c3a47a871f2194e54b2ee13a2029ee4..932e2afe81258a186dcfb31bbbb15f7bda0d67b9 100644 (file)
@@ -1,5 +1,11 @@
 2020-04-20  Jonathan Wakely  <jwakely@redhat.com>
 
+       * testsuite/20_util/is_constructible/51185.cc: Make test class a
+       non-aggregate so that the test verifies the same thing in all -std
+       modes.
+       * testsuite/20_util/is_constructible/value-2.cc: Adjust expected
+       results for some types when paren-init for aggregates is supported.
+
        * include/std/version (__cpp_lib_three_way_comparison): Update value.
        * libsupc++/compare (__cpp_lib_three_way_comparison): Likewise.
        (__detail::__synth3way): Add noexcept-specifier.
index d2750d8607e238e1cafd510937ce077b5f165ea4..8605135ad5a7cf1f8d60d1cbdc8710bd0d66c904 100644 (file)
 #include <type_traits>
 
 struct A { };
-struct B : A { };
+struct B : A {
+#if __cpp_aggregate_bases && __cpp_aggregate_paren_init
+  // A user-declared constructor prevents B from being an aggregate.
+  // Otherwise 'B&& b{A{}};' becomes valid in C++17 (__cpp_aggregate_bases),
+  // and 'B&& b(A{});' becomes valid in C++17 (__cpp_aggregate_paren_init).
+  B();
+#endif
+};
 
 // libstdc++/51185
 void f()
index c54b749c0463966a01e5979c03fae01e8cf98792..8ba50e1efe7c169e66c55fb30e5d240134cfcca4 100644 (file)
@@ -216,12 +216,20 @@ static_assert(std::is_constructible<const B&&, D&&>::value, "Error");
 static_assert(!std::is_constructible<B&, const D&>::value, "Error");
 static_assert(!std::is_constructible<B&&, const D&&>::value, "Error");
 
+#if __cpp_aggregate_bases && __cpp_aggregate_paren_init
+// In C++20 an rvalue reference or const lvalue reference can bind to a
+// temporary of aggregate type that is initialized from a base class value.
+constexpr bool v = true;
+#else
+constexpr bool v = false;
+#endif
+
 static_assert(!std::is_constructible<D&, B&>::value, "Error");
-static_assert(!std::is_constructible<D&&, B&&>::value, "Error");
+static_assert(v == std::is_constructible<D&&, B&&>::value, "Error");
 static_assert(!std::is_constructible<D&, const B&>::value, "Error");
-static_assert(!std::is_constructible<D&&, const B&&>::value, "Error");
-static_assert(!std::is_constructible<const D&, B&>::value, "Error");
-static_assert(!std::is_constructible<const D&&, B&&>::value, "Error");
+static_assert(v == std::is_constructible<D&&, const B&&>::value, "Error");
+static_assert(v == std::is_constructible<const D&, B&>::value, "Error");
+static_assert(v == std::is_constructible<const D&&, B&&>::value, "Error");
 
 static_assert(!std::is_constructible<B&&, B&>::value, "Error");
 static_assert(!std::is_constructible<B&&, D&>::value, "Error");
@@ -754,14 +762,21 @@ static_assert(!std::is_constructible<FromArgs<std::initializer_list<int>&,
              std::initializer_list<B>&>, std::initializer_list<int>,
              std::initializer_list<B>>::value, "Error");
 
+#if __cpp_aggregate_paren_init
+// In C++20 arrays can be initialized using parentheses.
+constexpr bool w = true;
+#else
+constexpr bool w = false;
+#endif
+
 static_assert(!std::is_constructible<FromArgs<std::initializer_list<int>>,
              int, int>::value, "Error");
 static_assert(!std::is_constructible<const
              FromArgs<std::initializer_list<int>>, int, int>::value, "Error");
-static_assert(!std::is_constructible<B[2], B, B>::value, "Error");
-static_assert(!std::is_constructible<const B[2], B, B>::value, "Error");
-static_assert(!std::is_constructible<U[2], U, U>::value, "Error");
-static_assert(!std::is_constructible<const U[2], U, U>::value, "Error");
+static_assert(w == std::is_constructible<B[2], B, B>::value, "Error");
+static_assert(w == std::is_constructible<const B[2], B, B>::value, "Error");
+static_assert(w == std::is_constructible<U[2], U, U>::value, "Error");
+static_assert(w == std::is_constructible<const U[2], U, U>::value, "Error");
 
 static_assert(!std::is_constructible<E, E, E>::value, "Error");
 static_assert(!std::is_constructible<const E, E, E>::value, "Error");