Skip to content

Commit 03ddb1b

Browse files
committed
[libc++][vector] include formatter only in C++23
This is to reduce included header file size in pre C++23 build. e.g. 2% of total included file coming from formatter_bool.h in chrome build. https://commondatastorage.googleapis.com/chromium-browser-clang/chrome_includes_2024-10-17_004801.html#view=edges&filter=&sort=asize&reverse=&includer=%5Ethird_party%2Flibc%5C%2B%5C%2B%2Fsrc%2Finclude%2Fvector%24&included=&limit=1000
1 parent 23d4fe6 commit 03ddb1b

File tree

1 file changed

+5
-2
lines changed

1 file changed

+5
-2
lines changed

libcxx/include/vector

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -324,8 +324,6 @@ template<class T, class charT> requires is-vector-bool-reference<T> // Since C++
324324
#include <__config>
325325
#include <__debug_utils/sanitizers.h>
326326
#include <__format/enable_insertable.h>
327-
#include <__format/formatter.h>
328-
#include <__format/formatter_bool.h>
329327
#include <__functional/hash.h>
330328
#include <__functional/unary_function.h>
331329
#include <__fwd/vector.h>
@@ -391,6 +389,11 @@ template<class T, class charT> requires is-vector-bool-reference<T> // Since C++
391389
#include <compare>
392390
#include <initializer_list>
393391

392+
#if _LIBCPP_STD_VER >= 23
393+
# include <__format/formatter.h>
394+
# include <__format/formatter_bool.h>
395+
#endif
396+
394397
#if !defined(_LIBCPP_HAS_NO_PRAGMA_SYSTEM_HEADER)
395398
# pragma GCC system_header
396399
#endif

0 commit comments

Comments
 (0)