[all-commits] [llvm/llvm-project] 409091: [libc++] Guard additional headers with _LIBCPP_HAS...

Louis Dionne via All-commits all-commits at lists.llvm.org
Fri Apr 4 08:49:08 PDT 2025


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 4090910a695efcba4b484e9f8ad2b564e9a4e7ed
      https://github.com/llvm/llvm-project/commit/4090910a695efcba4b484e9f8ad2b564e9a4e7ed
  Author: Louis Dionne <ldionne.2 at gmail.com>
  Date:   2025-04-04 (Fri, 04 Apr 2025)

  Changed paths:
    M libcxx/include/__configuration/platform.h
    M libcxx/include/__locale
    M libcxx/include/__locale_dir/locale_base_api.h
    M libcxx/include/fstream
    M libcxx/include/regex
    M libcxx/include/strstream
    M libcxx/test/configs/armv7m-picolibc-libc++.cfg.in
    M libcxx/test/libcxx/system_reserved_names.gen.py

  Log Message:
  -----------
  [libc++] Guard additional headers with _LIBCPP_HAS_LOCALIZATION (#131921)

There were some remaining headers that were not guarded with
_LIBCPP_HAS_LOCALIZATION, leading to errors when trying to use modules
on platforms that don't support localization (since all the headers get
pulled in when building the 'std' module). This patch brings these
headers in line with what we do for every other header that depends on
localization.

This patch also requires including <picolibc.h> from
<__configuration/platform.h> in order to define _NEWLIB_VERSION. In the
long term, we should use a better approach for doing that, such as
defining a macro in the __config_site header.



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications


More information about the All-commits mailing list