[PATCH] D78832: [Support] Fix fragile ThreadPool test

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Apr 25 11:08:28 PDT 2020


MaskRay accepted this revision.
MaskRay added a comment.

LGTM as well.



================
Comment at: llvm/unittests/Support/ThreadPool.cpp:188
     ThreadPool Pool(S);
-    for (size_t I = 0; I < 10000; ++I) {
+    for (size_t I = 0; I < S.compute_thread_count(); ++I) {
       Pool.async([&] {
----------------
Supposed this is deterministic now because of the `llvm::ThreadPoolStrategy::apply_thread_strategy` magic on Windows.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D78832





More information about the llvm-commits mailing list