[libcxx-commits] [PATCH] D149543: [libc++][format] Fixes vector<bool> requirements.

[Hans Wennborg via Phabricator via libcxx-commits]

hans added a comment.

> In D149543#4456497 <https://reviews.llvm.org/D149543#4456497>, @Mordante wrote:
>
>> 
>
> Interesting. Am I right to assume most of the new size is due to including `<string>`?

I don't think it's `<string>`; that's already widely included. I think it's `__format/formatter_output.h` where most of it gets added.

Part of me thinks maybe `<format>` could be gated behind a macro like is was while it was experimental, since we're not currently using it in Chromium. But that would only delay the issue until we do want to use it of course.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D149543