[PATCH] D54339: Fix threads detection on GNU/Hurd

Kamil Rytarowski via Phabricator reviews at reviews.llvm.org
Mon Nov 19 08:02:20 PST 2018


krytarowski added inline comments.


================
Comment at: include/__config:1146
       defined(__linux__) || \
+      defined(__GNU__) || \
       defined(__APPLE__) || \
----------------
ldionne wrote:
> According to https://sourceforge.net/p/predef/wiki/OperatingSystems/, an equivalent way of checking that would be `__gnu_hurd__`. I would prefer that as it is more explicit. Otherwise LGTM.
`__GNU__` is well established constant, used in the wild. I've never seen `__gnu_hurd__` myself, it souds like `__gnu_linux__`...


Repository:
  rCXX libc++

https://reviews.llvm.org/D54339





More information about the libcxx-commits mailing list