[libcxx-commits] [libcxx] [libc++] Use relocation in vector::emplace_back (PR #159365)

[Louis Dionne via libcxx-commits]

================
@@ -1188,19 +1188,43 @@ _LIBCPP_CONSTEXPR_SINCE_CXX20 inline
     void
 #endif
     vector<_Tp, _Allocator>::emplace_back(_Args&&... __args) {
-  pointer __end = this->__end_;
-  std::__if_likely_else(
-      __end < this->__cap_,
-      [&] {
-        __emplace_back_assume_capacity(std::forward<_Args>(__args)...);
-        ++__end;
-      },
-      [&] { __end = __emplace_back_slow_path(std::forward<_Args>(__args)...); });
-
-  this->__end_ = __end;
+  if constexpr (__libcpp_is_trivially_relocatable<value_type>::value &&
+                __allocator_has_trivial_move_construct_v<allocator_type, value_type> &&
----------------
ldionne wrote:

Food for thought: we might want to rename this to `__allocator_has_transparent_move_construct_v` & friends.

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