[PATCH] D41316: [libcxx] Allow random_device to be built optionally
Eric Fiselier via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Feb 9 18:17:09 PST 2018
EricWF added inline comments.
================
Comment at: test/std/experimental/filesystem/lit.local.cfg:5
+
+# filesystem_test_helper uses random_device to generate random path.
+if 'libcpp-has-no-random-device' in config.available_features:
----------------
EricWF wrote:
> EricWF wrote:
> > I would rather keep these tests running using a different source for the random seed if possible.
> Perhaps the random generator could be initialized using `std::chrono::system_clock::now().time_since_epoch().count()`, which should produce more distinct values that the typical `time(NULL)` method used to initialize `srand`.
Actually `std::chrono::high_resolution_clock` would be better.
Repository:
rCXX libc++
https://reviews.llvm.org/D41316
More information about the cfe-commits
mailing list