mehdi_amini added a comment.

There is something puzzling to me in this patch in the way the ThreadPool was 
modified: the client of the thread pool cannot request a number of threads 
anymore, but only a *maximum* number of threads. I don't really understand why? 
Using the OS reported concurrency was always a "default", isn't this change 
orthogonal / unrelated to the stated goal of this patch?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D71775



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

Reply via email to