[PATCH] D40319: [libcxx] Support getentropy as a source of randomness for std::random_device

[Jonathan Wakely via Phabricator via cfe-commits]

jwakely added inline comments.


================
Comment at: libcxx/trunk/src/random.cpp:29
+#if defined(_LIBCPP_USING_GETENTROPY)
+#include <sys/random.h>
+#elif defined(_LIBCPP_USING_DEV_RANDOM)
----------------
jwakely wrote:
> musl only declares `getentropy` in `<unistd.h>` not `<sys/random.h>`. Glibc declares it in both. Should `<unistd.h>` also be included when `_LIBCPP_USING_GETENTROPY` is defined, so it can work more portably?
I suppose it doesn't really matter, since `_LIBCPP_USING_GETENTROPY` is currently only defined for fuchsia and wasi, which both declare `getentropy` in `<sys/random.h>`.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D40319