[PATCH] D76885: [lld][ELF][WebAssembly] Replace --(no-)threads with --threads=N

[Fangrui Song via Phabricator via llvm-commits]

MaskRay created this revision.
MaskRay added reviewers: grimar, psmith, rnk, ruiu, sbc100.
Herald added subscribers: llvm-commits, wuzish, dexonsmith, sunfish, aheejin, jgravelle-google, arichardson, emaste, dschuff.
Herald added a reviewer: espindola.
Herald added a project: LLVM.
MaskRay updated this revision to Diff 252964.
MaskRay added a comment.
MaskRay updated this revision to Diff 252966.

Don't move threads in wasm/Options.td


MaskRay added a comment.

Fix COFF/Driver.cpp


--no-threads is a name copied from gold.
gold has --no-thread, --thread-count and several other --thread-count-*.

There are needs to customize the number of threads (running several lld
processes concurrently or customizing LTO threads).  Having a single
--threads=N is straightforward replacement of gold's --no-threads +
--thread-count.

--no-threads is used rarely. So just delete --no-threads instead of
keeping it for compatibility for a while.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D76885

Files:
  lld/COFF/Driver.cpp
  lld/Common/Filesystem.cpp
  lld/Common/Threads.cpp
  lld/ELF/Driver.cpp
  lld/ELF/ICF.cpp
  lld/ELF/Options.td
  lld/ELF/SyntheticSections.cpp
  lld/include/lld/Common/Threads.h
  lld/test/ELF/build-id.s
  lld/wasm/Driver.cpp
  lld/wasm/Options.td

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D76885.252966.patch
Type: text/x-patch
Size: 8485 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200326/d72dc485/attachment.bin>