[PATCH] D92271: Implement computeHostNumHardwareThreads() for FreeBSD

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Nov 28 12:24:15 PST 2020


MaskRay added inline comments.


================
Comment at: llvm/lib/Support/Unix/Threading.inc:30
+#if defined(__FreeBSD__)
+#include <sys/cpuset.h>
+#endif
----------------
Should this be merged with the following block?


================
Comment at: llvm/lib/Support/Unix/Threading.inc:295
+    return CPU_COUNT(&mask);
+#endif
 #ifdef __linux__
----------------
Use `#elif`


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D92271



More information about the llvm-commits mailing list