[libcxx-commits] [libcxx] [libc++] Refactor some code in monotonic_buffer_resource (PR #117271)

[Louis Dionne via libcxx-commits]

================
@@ -60,6 +60,18 @@ Version 20.0
   Symbol added: __ZTSNSt3__120__codecvt_utf8_utf16IDsEE
   Symbol added: __ZTSNSt3__120__codecvt_utf8_utf16IwEE
 
+* [libc++][PMR] Remove duplicate implementation of __try_allocate_from_chunk
+
+  This patch removes member function __try_allocate_from_chunk from __chunk_footer and __initial_descriptor.
+  The two implementation are almost same and can be easily moved to `memory_resource.cpp` inner implementation.
+  They are defined and only used in `memory_resource.cpp`, No necessary export to dylib
----------------
ldionne wrote:

```suggestion
  This patch removes member functions __try_allocate_from_chunk from __chunk_footer and __initial_descriptor.
  The two implementations were almost identical and can be easily moved to be implementation details of the built library. Since they were only used from within the built library, the removal of these symbols should not be an ABI break.
```

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