STL_MSFT marked an inline comment as done.
STL_MSFT added a comment.

Added the requested comment, will commit.



================
Comment at: 
test/std/algorithms/alg.modifying.operations/alg.random.sample/sample.pass.cpp:74
   assert(end.base() - oa == std::min(os, is));
-  assert(std::equal(oa, oa + os, oa1));
+  LIBCPP_ASSERT(std::equal(oa, oa + os, oa1));
   end = std::sample(PopulationIterator(ia),
----------------
EricWF wrote:
> Please add a comment here explaining that the algorithm is non-reproducable.
Adding:
  // sample() is deterministic but non-reproducible;
  // its results can vary between implementations.



https://reviews.llvm.org/D26816



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to