[libcxx-commits] [libcxx] [libc++] Implement Resolution of LWG 3886 (PR #155356)

[William Tran-Viet via libcxx-commits]

================
@@ -730,7 +730,8 @@ public:
             enable_if_t<_CheckOptionalArgsCtor<_Up>::template __enable_implicit<_Up>(), int> = 0>
   _LIBCPP_HIDE_FROM_ABI constexpr optional(_Up&& __v) : __base(in_place, std::forward<_Up>(__v)) {}
 
-  template <class _Up, enable_if_t<_CheckOptionalArgsCtor<_Up>::template __enable_explicit<_Up>(), int> = 0>
+  template <class _Up                                                                        = remove_cv_t<_Tp>,
----------------
smallp-o-p wrote:

I think there are tests for `{}` assignment, the other ones not so much--I'll get on those.

If you're referring to brace initialization and assignment that is.

https://github.com/llvm/llvm-project/pull/155356