[libcxx-commits] [libcxx] [libc++][vector] include formatter only in C++23 (PR #112675)
Takuto Ikuta via libcxx-commits
libcxx-commits at lists.llvm.org
Thu Oct 17 01:13:34 PDT 2024
https://github.com/atetubou created https://github.com/llvm/llvm-project/pull/112675
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
>From 03ddb1b85c6abd6766b4beb95b6749b68c4f7220 Mon Sep 17 00:00:00 2001
From: Takuto Ikuta <tikuta at google.com>
Date: Tue, 15 Oct 2024 08:30:23 +0000
Subject: [PATCH] [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
---
libcxx/include/vector | 7 +++++--
1 file changed, 5 insertions(+), 2 deletions(-)
diff --git a/libcxx/include/vector b/libcxx/include/vector
index dc31f31838264c..93755afdde711c 100644
--- a/libcxx/include/vector
+++ b/libcxx/include/vector
@@ -324,8 +324,6 @@ template<class T, class charT> requires is-vector-bool-reference<T> // Since C++
#include <__config>
#include <__debug_utils/sanitizers.h>
#include <__format/enable_insertable.h>
-#include <__format/formatter.h>
-#include <__format/formatter_bool.h>
#include <__functional/hash.h>
#include <__functional/unary_function.h>
#include <__fwd/vector.h>
@@ -391,6 +389,11 @@ template<class T, class charT> requires is-vector-bool-reference<T> // Since C++
#include <compare>
#include <initializer_list>
+#if _LIBCPP_STD_VER >= 23
+# include <__format/formatter.h>
+# include <__format/formatter_bool.h>
+#endif
+
#if !defined(_LIBCPP_HAS_NO_PRAGMA_SYSTEM_HEADER)
# pragma GCC system_header
#endif
More information about the libcxx-commits
mailing list