[libcxx-commits] [PATCH] D138512: [libc++] Add missing __has_include checks for C headers not provided by libc++

Nikolas Klauser via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Tue Nov 22 11:07:57 PST 2022


philnik accepted this revision.
philnik added a comment.
This revision is now accepted and ready to land.

LGTM with green CI.



================
Comment at: libcxx/include/cassert:25
+#  include <assert.h>
+#endif
 
----------------
Could we add something like
```lang=c++
#ifdef _LIBCPP_ASSERT_H
#  error "Does libc++ provide an assert.h now?"
#endif
```
to make sure?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D138512



More information about the libcxx-commits mailing list