[PATCH] D59254: [RFC] Implementation of Clang randstruct
Connor Kuehl via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Mar 13 07:25:47 PDT 2019
connorkuehl added inline comments.
Comment at: clang/include/clang/AST/RecordFieldReorganizer.h:54
+ std::seed_seq Seq;
+ std::default_random_engine rng;
> I don't think we can use `default_random_engine` for this because the behaviour would need to be consistent between C++ standard library implementations, and the behaviour of `default_random_engine` is implementation defined. Similarly, I don't think that we can use `std::shuffle` (see note in https://en.cppreference.com/w/cpp/algorithm/random_shuffle ).
Sure thing, we'll begin investigating alternatives.
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
More information about the cfe-commits