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

[Louis Dionne via All-commits]

  Branch: refs/heads/release/20.x
  Home:   https://github.com/llvm/llvm-project
  Commit: dfd6f123362acc94eb99c4e0440fa166b40153e4
      https://github.com/llvm/llvm-project/commit/dfd6f123362acc94eb99c4e0440fa166b40153e4
  Author: Louis Dionne <ldionne.2 at gmail.com>
  Date:   2025-04-15 (Tue, 15 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.

(cherry picked from commit 4090910a695efcba4b484e9f8ad2b564e9a4e7ed)



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