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

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 1 12:36:57 PDT 2020


MaskRay marked an inline comment as done.
MaskRay added a comment.

In D76885#1955404 <https://reviews.llvm.org/D76885#1955404>, @smeenai wrote:

> For people who were using `-Wl,--no-threads` to disable parallelism in (some parts of) their build system, what's the way forward if they want to support both older and newer LLD versions?
>
> (I suppose feature flag detection is one way, but that's not really a thing on some build systems, including Bazel, I believe.)


-Wl,--no-threads should be very rare. https://github.com/halide/Halide/blob/master/src/runtime/hexagon_remote/Makefile and libsodium_1.0.18-1/dist-build/wasm32-wasi.sh are the only open source cases I can find. You probably should make the change while bumping the version of lld.


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