[libcxx-commits] [PATCH] D148826: [libc++][format] Removes vector dependency.

Mark de Wever via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Sat Apr 22 03:31:07 PDT 2023


Mordante created this revision.
Herald added a project: All.
Mordante published this revision for review.
Herald added a project: libc++.
Herald added a subscriber: libcxx-commits.
Herald added a reviewer: libc++.

During the review of D140653 <https://reviews.llvm.org/D140653> it was suggested to use vector in
__retarget_buffer instead of manually managing the memory. Due to the
requirements of the Standard it turns out format needs to include vector
leading to a cycle. Therefore switching back to manual memory
management.

This is a preparation to fix https://llvm.org/PR61314


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D148826

Files:
  libcxx/include/__format/buffer.h
  libcxx/test/libcxx/transitive_includes/cxx03.csv
  libcxx/test/libcxx/transitive_includes/cxx11.csv
  libcxx/test/libcxx/transitive_includes/cxx14.csv
  libcxx/test/libcxx/transitive_includes/cxx17.csv
  libcxx/test/libcxx/transitive_includes/cxx20.csv
  libcxx/test/libcxx/transitive_includes/cxx2b.csv

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D148826.515417.patch
Type: text/x-patch
Size: 14392 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/libcxx-commits/attachments/20230422/883cd833/attachment.bin>


More information about the libcxx-commits mailing list