summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
-rw-r--r--libcxx/test/std/containers/sequences/array/array.cons/deduct.pass.cpp6
-rw-r--r--libcxx/test/std/utilities/optional/optional.object/optional.object.ctor/deduct.fail.cpp10
-rw-r--r--libcxx/test/std/utilities/optional/optional.object/optional.object.ctor/deduct.pass.cpp13
3 files changed, 17 insertions, 12 deletions
diff --git a/libcxx/test/std/containers/sequences/array/array.cons/deduct.pass.cpp b/libcxx/test/std/containers/sequences/array/array.cons/deduct.pass.cpp
index faa6223bc92..0f41c7b811a 100644
--- a/libcxx/test/std/containers/sequences/array/array.cons/deduct.pass.cpp
+++ b/libcxx/test/std/containers/sequences/array/array.cons/deduct.pass.cpp
@@ -9,7 +9,10 @@
// <array>
// UNSUPPORTED: c++98, c++03, c++11, c++14
+// UNSUPPORTED: clang-5
// UNSUPPORTED: libcpp-no-deduction-guides
+// Clang 5 will generate bad implicit deduction guides
+// Specifically, for the copy constructor.
// template <class T, class... U>
@@ -51,8 +54,6 @@ int main()
}
// Test the implicit deduction guides
-// FIXME broken: no matching constructor
-#if 0
{
std::array<double, 2> source = {4.0, 5.0};
std::array arr(source); // array(array)
@@ -61,5 +62,4 @@ int main()
assert(arr[0] == 4.0);
assert(arr[1] == 5.0);
}
-#endif
}
diff --git a/libcxx/test/std/utilities/optional/optional.object/optional.object.ctor/deduct.fail.cpp b/libcxx/test/std/utilities/optional/optional.object/optional.object.ctor/deduct.fail.cpp
index 5bbb77d46b2..b76b730d274 100644
--- a/libcxx/test/std/utilities/optional/optional.object/optional.object.ctor/deduct.fail.cpp
+++ b/libcxx/test/std/utilities/optional/optional.object/optional.object.ctor/deduct.fail.cpp
@@ -9,7 +9,10 @@
// <optional>
// UNSUPPORTED: c++98, c++03, c++11, c++14
+// UNSUPPORTED: clang-5
// UNSUPPORTED: libcpp-no-deduction-guides
+// Clang 5 will generate bad implicit deduction guides
+// Specifically, for the copy constructor.
// template<class T>
@@ -28,7 +31,12 @@ int main()
// Test the implicit deduction guides
{
// optional()
- std::optional opt; // expected-error {{declaration of variable 'opt' with deduced type 'std::optional' requires an initializer}}
+ std::optional opt; // expected-error-re {{{{declaration of variable 'opt' with deduced type 'std::optional' requires an initializer|no viable constructor or deduction guide for deduction of template arguments of 'optional'}}}}
+// clang-6 gives a bogus error here:
+// declaration of variable 'opt' with deduced type 'std::optional' requires an initializer
+// clang-7 (and later) give a better message:
+// no viable constructor or deduction guide for deduction of template arguments of 'optional'
+// So we check for one or the other.
}
{
diff --git a/libcxx/test/std/utilities/optional/optional.object/optional.object.ctor/deduct.pass.cpp b/libcxx/test/std/utilities/optional/optional.object/optional.object.ctor/deduct.pass.cpp
index 807055eed44..0a5f45668cf 100644
--- a/libcxx/test/std/utilities/optional/optional.object/optional.object.ctor/deduct.pass.cpp
+++ b/libcxx/test/std/utilities/optional/optional.object/optional.object.ctor/deduct.pass.cpp
@@ -9,7 +9,10 @@
// <optional>
// UNSUPPORTED: c++98, c++03, c++11, c++14
+// UNSUPPORTED: clang-5
// UNSUPPORTED: libcpp-no-deduction-guides
+// Clang 5 will generate bad implicit deduction guides
+// Specifically, for the copy constructor.
// template<class T>
@@ -40,18 +43,12 @@ int main()
}
// Test the implicit deduction guides
-
{
-// optional(const optional &);
- // FIXME clang and GCC disagree about this!
- // clang thinks opt is optional<optional<char>>, GCC thinks it's optional<char>.
-#if 0
+// optional(optional);
std::optional<char> source('A');
std::optional opt(source);
- static_assert(std::is_same_v<decltype(opt), std::optional<std::optional<char>>>, "");
+ static_assert(std::is_same_v<decltype(opt), std::optional<char>>, "");
assert(static_cast<bool>(opt) == static_cast<bool>(source));
assert(*opt == *source);
-#endif
}
-
}
OpenPOWER on IntegriCloud