[PATCH] D75153: [ThinLTO] Allow usage of all SMT threads in the system
Alex Brachet via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Mar 4 18:31:26 PST 2020
abrachet added inline comments.
================
Comment at: llvm/include/llvm/Support/Threading.h:201
+ /// hardware core is used.
+ inline ThreadPoolStrategy heavyweight_hardware_concurrency(StringRef Num) {
+ Optional<ThreadPoolStrategy> S =
----------------
Nit: Remove `inline` https://llvm.org/docs/CodingStandards.html#don-t-use-inline-when-defining-a-function-in-a-class-definition
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D75153/new/
https://reviews.llvm.org/D75153
More information about the cfe-commits
mailing list