[libcxx-commits] [libcxx] e6be5c9 - [libc++] Move _LIBCPP_FOPEN_CLOEXEC_MODE to <fstream> (#205537)

[via libcxx-commits]

Author: Nikolas Klauser
Date: 2026-06-26T18:30:54+02:00
New Revision: e6be5c920d5074792a63b9a2d88904fc682d7a24

URL: https://github.com/llvm/llvm-project/commit/e6be5c920d5074792a63b9a2d88904fc682d7a24
DIFF: https://github.com/llvm/llvm-project/commit/e6be5c920d5074792a63b9a2d88904fc682d7a24.diff

LOG: [libc++] Move _LIBCPP_FOPEN_CLOEXEC_MODE to <fstream> (#205537)

The macro is only required inside `<fstream>`, so we can move it there
instead of having it as a general configuration macro.

Added: 
    

Modified: 
    libcxx/include/__config
    libcxx/include/fstream

Removed: 
    


################################################################################
diff  --git a/libcxx/include/__config b/libcxx/include/__config
index fc7a121d52783..4ec44befe6d13 100644
--- a/libcxx/include/__config
+++ b/libcxx/include/__config
@@ -357,17 +357,6 @@ typedef __char32_t char32_t;
 #    endif // defined(_LIBCPP_ABI_MICROSOFT) && !defined(_LIBCPP_BUILDING_LIBRARY)
 #  endif   // _LIBCPP_NO_AUTO_LINK
 
-// Configures the fopen close-on-exec mode character, if any. This string will
-// be appended to any mode string used by fstream for fopen/fdopen.
-//
-// Not all platforms support this, but it helps avoid fd-leaks on platforms that
-// do.
-#  if defined(__BIONIC__)
-#    define _LIBCPP_FOPEN_CLOEXEC_MODE "e"
-#  else
-#    define _LIBCPP_FOPEN_CLOEXEC_MODE
-#  endif
-
 // c8rtomb() and mbrtoc8() were added in C++20 and C23. Support for these
 // functions is gradually being added to existing C libraries. The conditions
 // below check for known C library versions and conditions under which these

diff  --git a/libcxx/include/fstream b/libcxx/include/fstream
index b5fb65820628a..0c34c9149de0d 100644
--- a/libcxx/include/fstream
+++ b/libcxx/include/fstream
@@ -600,6 +600,17 @@ inline bool basic_filebuf<_CharT, _Traits>::is_open() const {
   return __file_ != nullptr;
 }
 
+// Configures the fopen close-on-exec mode character, if any. This string will
+// be appended to any mode string used by fstream for fopen/fdopen.
+//
+// Not all platforms support this, but it helps avoid fd-leaks on platforms that
+// do.
+#    if defined(__BIONIC__)
+#      define _LIBCPP_FOPEN_CLOEXEC_MODE "e"
+#    else
+#      define _LIBCPP_FOPEN_CLOEXEC_MODE
+#    endif
+
 template <class _CharT, class _Traits>
 const char* basic_filebuf<_CharT, _Traits>::__make_mdstring(ios_base::openmode __mode) _NOEXCEPT {
   switch (__mode & ~ios_base::ate) {