[all-commits] [llvm/llvm-project] 23e1ed: [libc++] Split up ABI and platform configuration t...

Louis Dionne via All-commits all-commits at lists.llvm.org
Tue May 28 04:22:29 PDT 2024


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 23e1ed65c2c3eb1f80f7eeb4897ec843492c5451
      https://github.com/llvm/llvm-project/commit/23e1ed65c2c3eb1f80f7eeb4897ec843492c5451
  Author: Louis Dionne <ldionne.2 at gmail.com>
  Date:   2024-05-28 (Tue, 28 May 2024)

  Changed paths:
    M libcxx/include/CMakeLists.txt
    M libcxx/include/__config
    A libcxx/include/__configuration/abi.h
    A libcxx/include/__configuration/compiler.h
    A libcxx/include/__configuration/platform.h
    M libcxx/include/module.modulemap
    M libcxx/utils/generate_iwyu_mapping.py

  Log Message:
  -----------
  [libc++] Split up ABI and platform configuration to their own headers (#90863)

This is a first step towards splitting up the <__config> header. The
<__config> header is large and rather disorganized at this point,
leading to confusion and subtle mistakes. For example, we never noticed
that the string layout used on arm64 was only enabled for the Clang
compiler, as the setting being in the compiler == clang block was
probably never intentional.

The danger of splitting up the <__config> header is to implicitly use
undefined macros that should have been defined prior to their usage,
however this can be remediated with -Wundef and we've started moving
towards -Wundef enforceable macros.



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