[PATCH] D78408: [llvm-cov] Prevent llvm-cov from using too many threads

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Apr 18 00:30:57 PDT 2020


MaskRay added a comment.

I think we should respect ThreadsRequested if it is greater than 0.

For the particular llvm-cov issue, we can probably restore the previous behavior:

  NumThreads =
      std::max(1U, std::min(llvm::heavyweight_hardware_concurrency(),
                            unsigned(SourceFiles.size())));
  NumThreads = SourceFiles.size();


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D78408





More information about the llvm-commits mailing list