[PATCH] D54338: Fix threads build on GNU/Hurd

Louis Dionne via Phabricator reviews at reviews.llvm.org
Sun Nov 11 19:12:38 PST 2018


ldionne requested changes to this revision.
ldionne added a comment.
This revision now requires changes to proceed.

I’m OK with the change but please don’t repeat the condition on L24.

Re-upload and I will commit.



================
Comment at: src/thread.cpp:24
 #   include <sys/sysctl.h>
-# endif // defined(BSD)
+# endif // defined(__FreeBSD__) || defined(__NetBSD__) || defined(__OpenBSD__) || defined(__DragonFly__) || defined(__APPLE__)
 #endif // defined(__unix__) || (defined(__APPLE__) && defined(__MACH__))
----------------
I really don’t feel like it is useful to repeat the condition given the #if spans only one line.


Repository:
  rCXX libc++

https://reviews.llvm.org/D54338





More information about the libcxx-commits mailing list