mirror of
https://github.com/llvm/llvm-project.git
synced 2025-04-25 03:06:04 +00:00
ba87515fea
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>`.
The code in this directory is based on Ulf Adams's work. The upstream for the code is: https://github.com/ulfjack/ryu/tree/59661c3/ryu The code has been adapted by Stephan T. Lavavej of Microsoft for usage in std::to_chars. This code has been contributed by Microsoft for inclusion in libc++. The code in this directory has a different coding style than other parts to minimize the number of changes by both upstream sources.