rsmith added inline comments.
================
Comment at: include/__config:153
+#ifndef __has_include
+#define __has_include(...) 0
#endif
----------------
arichardson wrote:
> EricWF wrote:
> > I do prefer not hijacking this name, but if it's needed to make things
> > work, then it's OK with me.
> >
> > @mclow.lists Are you OK if we steal this identifier and `#define` it
> > ourselves.
> This is the correct way of dealing with missing `__has_include()` according
> to @rsmith
If you would prefer to not define this name yourself, the other option would be
to replace all uses of `__has_include(blah)` with
```
#ifdef __has_include
#if __has_include(blah)
//...
#endif
#endif
```
================
Comment at: include/__config:1062-1075
# if defined(__FreeBSD__) || \
defined(__Fuchsia__) || \
defined(__NetBSD__) || \
defined(__linux__) || \
defined(__APPLE__) || \
defined(__CloudABI__) || \
defined(__sun__) || \
----------------
Example:
```
#if defined(__MINGW32__) && defined(__has_include)
#if __has_include(<pthread.h>)
#define _LIBCPP_MINGW32_HAS_PTHREAD
#endif
#endif
#if defined(__FreeBSD__) || \
// [...]
defined(__sun__) || \
defined(_LIBCPP_MINGW32_HAS_PTHREAD)
```
Repository:
rCXX libc++
https://reviews.llvm.org/D49067
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
