STL_MSFT added a comment. Otherwise, looks cromulent to me.
================ Comment at: test/std/utilities/optional/optional.nullopt/nullopt_t.pass.cpp:38 - static_assert(test(nullopt) == 3, ""); + static_assert(std::is_same_v<const nullopt_t, decltype(std::nullopt)>); + static_assert(test()); ---------------- You're saying `std::nullopt` but you already dragged it in. https://reviews.llvm.org/D37024 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
