diff options
| author | Nikolas Klauser <nikolasklauser@berlin.de> | 2024-10-12 09:49:52 +0200 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2024-10-12 09:49:52 +0200 |
| commit | ba87515fea90b5d55836a8e3be63a7e683ce299d (patch) | |
| tree | 25b36e09d01ff31f453e6a8141629a52a7f673bb /llvm/lib/Bitcode/Reader/BitcodeReader.cpp | |
| parent | dbd197118db597970a5a9c5688c5e0bb01948ebb (diff) | |
| download | llvm-ba87515fea90b5d55836a8e3be63a7e683ce299d.zip llvm-ba87515fea90b5d55836a8e3be63a7e683ce299d.tar.gz llvm-ba87515fea90b5d55836a8e3be63a7e683ce299d.tar.bz2 | |
[libc++][RFC] Always define internal feature test macros (#89178)
Currently, the library-internal feature test macros are only defined if
the feature is not available, and always have the prefix
`_LIBCPP_HAS_NO_`. This patch changes that, so that they are always
defined and have the prefix `_LIBCPP_HAS_` instead. This changes the
canonical use of these macros to `#if _LIBCPP_HAS_FEATURE`, which means
that using an undefined macro (e.g. due to a missing include) is
diagnosed now. While this is rather unlikely currently, a similar change
in `<__configuration/availability.h>` caught a few bugs. This also
improves readability, since it removes the double-negation of `#ifndef
_LIBCPP_HAS_NO_FEATURE`.
The current patch only touches the macros defined in `<__config>`. If
people are happy with this approach, I'll make a follow-up PR to also
change the macros defined in `<__config_site>`.
Diffstat (limited to 'llvm/lib/Bitcode/Reader/BitcodeReader.cpp')
0 files changed, 0 insertions, 0 deletions