[PATCH] D147493: [ELF] Cap parallel::strategy to 16 threads when --threads= is unspecified

[Andrew Ng via Phabricator via llvm-commits]

andrewng added a comment.

Sorry, I was so focused on the value of the limit that I didn't really look at the details of the code change. Was it deliberate to have `config->threadCount` not match the number of threads in the `parallel::strategy`? IIUC, this probably doesn't affect correctness but might have an impact on efficiency.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D147493