[PATCH] D76885: [lld][COFF][ELF][WebAssembly] Replace --[no-]threads /threads[:no] with --threads={all,1,2,...} /threads:{all,1,2,...}

Alexandre Ganea via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 30 10:16:36 PDT 2020


aganea added inline comments.


================
Comment at: llvm/lib/Support/Parallel.cpp:137
+
+  struct Creator {
+    static void *call() { return new ThreadPoolExecutor(strategy); }
----------------
Do you think you can move this inside the `ThreadPoolExecutor` class, like `ThreadPoolExecutor::Deleter`?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D76885





More information about the llvm-commits mailing list