From 649da70defac4ce1f2236cec03d0710cfccaac12 Mon Sep 17 00:00:00 2001 From: Eric Fiselier Date: Sun, 17 Jul 2016 20:32:30 +0000 Subject: [PATCH] Fix std::experimental::optional tests. Patch from Casey Carter. llvm-svn: 275732 --- .../emplace_initializer_list.pass.cpp | 2 +- .../optional.object.ctor/initializer_list.pass.cpp | 12 +++--------- .../optional/optional.specalg/make_optional.pass.cpp | 4 +++- .../optional.syn/optional_const_nullopt_t.fail.cpp | 2 +- .../optional.syn/optional_nullopt_t.fail.cpp | 2 +- 5 files changed, 9 insertions(+), 13 deletions(-) diff --git a/libcxx/test/std/experimental/optional/optional.object/optional.object.assign/emplace_initializer_list.pass.cpp b/libcxx/test/std/experimental/optional/optional.object/optional.object.assign/emplace_initializer_list.pass.cpp index bb8fcd5a7ce4..fec37408e43b 100644 --- a/libcxx/test/std/experimental/optional/optional.object/optional.object.assign/emplace_initializer_list.pass.cpp +++ b/libcxx/test/std/experimental/optional/optional.object/optional.object.assign/emplace_initializer_list.pass.cpp @@ -59,7 +59,7 @@ public: static bool dtor_called; constexpr Z() : i_(0) {} constexpr Z(int i) : i_(i) {} - constexpr Z(std::initializer_list il) : i_(il.begin()[0]), j_(il.begin()[1]) + Z(std::initializer_list il) : i_(il.begin()[0]), j_(il.begin()[1]) {throw 6;} ~Z() {dtor_called = true;} diff --git a/libcxx/test/std/experimental/optional/optional.object/optional.object.ctor/initializer_list.pass.cpp b/libcxx/test/std/experimental/optional/optional.object/optional.object.ctor/initializer_list.pass.cpp index acc139ed9b5a..9bd6b18989fc 100644 --- a/libcxx/test/std/experimental/optional/optional.object/optional.object.ctor/initializer_list.pass.cpp +++ b/libcxx/test/std/experimental/optional/optional.object/optional.object.ctor/initializer_list.pass.cpp @@ -59,7 +59,7 @@ class Z public: constexpr Z() : i_(0) {} constexpr Z(int i) : i_(i) {} - constexpr Z(std::initializer_list il) : i_(il.begin()[0]), j_(il.begin()[1]) + Z(std::initializer_list il) : i_(il.begin()[0]), j_(il.begin()[1]) {throw 6;} friend constexpr bool operator==(const Z& x, const Z& y) @@ -97,6 +97,8 @@ int main() : optional(in_place, i) {} }; + constexpr test_constexpr_ctor dopt(in_place, {42, 101, -1}); + static_assert(*dopt == Y{42, 101, -1}, ""); } { static_assert(std::is_constructible, std::initializer_list&>::value, ""); @@ -109,13 +111,5 @@ int main() { assert(i == 6); } - - struct test_constexpr_ctor - : public optional - { - constexpr test_constexpr_ctor(in_place_t, std::initializer_list i) - : optional(in_place, i) {} - }; - } } diff --git a/libcxx/test/std/experimental/optional/optional.specalg/make_optional.pass.cpp b/libcxx/test/std/experimental/optional/optional.specalg/make_optional.pass.cpp index 7f5ffea6fa7a..9abd87bd4051 100644 --- a/libcxx/test/std/experimental/optional/optional.specalg/make_optional.pass.cpp +++ b/libcxx/test/std/experimental/optional/optional.specalg/make_optional.pass.cpp @@ -20,6 +20,8 @@ #include #include +#include "test_macros.h" + int main() { using std::experimental::optional; @@ -38,7 +40,7 @@ int main() std::string s("123"); optional opt = make_optional(std::move(s)); assert(*opt == "123"); - assert(s.empty()); + LIBCPP_ASSERT(s.empty()); } { std::unique_ptr s(new int(3)); diff --git a/libcxx/test/std/experimental/optional/optional.syn/optional_const_nullopt_t.fail.cpp b/libcxx/test/std/experimental/optional/optional.syn/optional_const_nullopt_t.fail.cpp index fb695d298206..89c207306aba 100644 --- a/libcxx/test/std/experimental/optional/optional.syn/optional_const_nullopt_t.fail.cpp +++ b/libcxx/test/std/experimental/optional/optional.syn/optional_const_nullopt_t.fail.cpp @@ -11,7 +11,7 @@ // // A program that necessitates the instantiation of template optional for -// (possibly cv-qualified) null_opt_t is ill-formed. +// (possibly cv-qualified) nullopt_t is ill-formed. #include diff --git a/libcxx/test/std/experimental/optional/optional.syn/optional_nullopt_t.fail.cpp b/libcxx/test/std/experimental/optional/optional.syn/optional_nullopt_t.fail.cpp index 9b62c9b39949..3d276d642022 100644 --- a/libcxx/test/std/experimental/optional/optional.syn/optional_nullopt_t.fail.cpp +++ b/libcxx/test/std/experimental/optional/optional.syn/optional_nullopt_t.fail.cpp @@ -11,7 +11,7 @@ // // A program that necessitates the instantiation of template optional for -// (possibly cv-qualified) null_opt_t is ill-formed. +// (possibly cv-qualified) nullopt_t is ill-formed. #include