[PATCH] D58594: [LLD][COFF] Support /threads[:no] like the ELF driver

Reid Kleckner via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 25 11:05:26 PST 2019


rnk accepted this revision.
rnk added a comment.
This revision is now accepted and ready to land.

lgtm

At first I was thinking, maybe we should extend this to /threads:N, so some build systems could reduce the parallelism when links run concurrently, but I think for Windows it's best to just let concrt/the system do its thing.

The other big source of expensive concurrency is ThinLTO, which logically we should wire up to /cgthreads.


Repository:
  rLLD LLVM Linker

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

https://reviews.llvm.org/D58594





More information about the llvm-commits mailing list