[libcxx-commits] [PATCH] D90257: [libc++] Use the using_if_exists attribute when provided

Louis Dionne via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Tue Oct 27 11:54:25 PDT 2020


ldionne added a comment.

In D90257#2357223 <https://reviews.llvm.org/D90257#2357223>, @lebedev.ri wrote:

> In D90257#2357188 <https://reviews.llvm.org/D90257#2357188>, @ldionne wrote:
>
>> Note that once this is in, we can also start dropping existing `#if`s in the headers to workaround various declarations missing in the underlying C stdlib, as long as we require a sufficiently recent Clang.
>
> What about other compilers?

We're not there yet -- I was just pointing to a possible future cleanup.

For other compilers: it seems reasonable to mandate that the attribute is supported on the compiler you're using if you're trying to port libc++ to a platform that doesn't provide a full standard library. It's a balance between ease-of-porting and ease-of-maintaining.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D90257/new/

https://reviews.llvm.org/D90257



More information about the libcxx-commits mailing list