[libcxx-commits] [PATCH] D154286: [libc++][format] Granularize formatter_output.

[Mark de Wever via Phabricator via libcxx-commits]

Mordante added a comment.

In D154286#4485094 <https://reviews.llvm.org/D154286#4485094>, @ldionne wrote:

> In D154286#4482802 <https://reviews.llvm.org/D154286#4482802>, @Mordante wrote:
>
>> @hans can you test whether this solves your issue. I expect it does not, but it would be nice to get confirmation.
>>
>> I don't think I can reduce the since of `vector` further by granularizing. If that is the case we need to consider conditionally including `<__format/*.h>` from `<vector>`. The formatter for `vector` is available in C++23 and later.
>
> In that case I think we should open a LWG issue to at least raise awareness about this issue. That's also a good way to check whether other implementers have noticed a similar problem.

I doubt the other implementers have noticed this; neither MSVC STL nor libstdc++ have implemented P2286 <https://reviews.llvm.org/P2286>, which introduced this dependency.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D154286/new/

https://reviews.llvm.org/D154286