[PATCH] D75153: [ThinLTO] Allow usage of all SMT threads in the system

Alex Brachet via Phabricator via llvm-commits llvm-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 llvm-commits mailing list