copy_assign.pass.cpp

Author: vinay-deshmukh

libcxx/include/__tree

@@ -1217,23 +1217,22 @@ private:
   __assign_value( __get_node_value_type_t<value_type>& __lhs, _From&& __rhs) {
     using __key_type = __remove_const_t<typename value_type::first_type>;
 
-    if (
-      (
-        _LIBCPP_STD_VER >= 26
-      )
-      && 
-      __libcpp_is_constant_evaluated()) {
+      
+    #if _LIBCPP_STD_VER >= 26
+    if (__libcpp_is_constant_evaluated()) {
       using __node_value_type = __get_node_value_type_t<value_type>;
       __get_node_value_type_t<value_type> __tmp(__rhs.first, __rhs.second);
       // const_cast is not allowed at constexpr time.
       // we get around this by deleting and creating a new node value
       // note: this does not work for `const MoveOnly` keys at constexpr time
-      __lhs.~__node_value_type();
+
+      // __lhs.~__node_value_type();
       __node_allocator& __na = __node_alloc();
       __node_traits::destroy(__na, std::addressof(__lhs));
-      // Fails here for `std/containers/associative/map/map.cons/move_assign.pass.cpp`
       __node_traits::construct(__na, std::addressof(__lhs), std::move(__tmp));
-    } else {
+    } else 
+    #endif
+    {
       // This is technically UB, since the object was constructed as `const`.
       // Clang doesn't optimize on this currently though.
       const_cast<__key_type&>(__lhs.first) = const_cast<__copy_cvref_t<_From, __key_type>&&>(__rhs.first);