diff options
| author | Jacob Pratt <jacob@jhpratt.dev> | 2025-03-14 01:37:33 -0400 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2025-03-14 01:37:33 -0400 |
| commit | e928a8f4a05c63ca5e9031741b0ee51e2254f92d (patch) | |
| tree | fd8fadac1443e75003a4da9b8045a94fc46d4dee | |
| parent | 91e4bab25ff9f37bbb28aa4d2d80feece1e52a49 (diff) | |
| parent | c8a6fcc3c838af8152fda0a22a8946914aaa5770 (diff) | |
| download | rust-e928a8f4a05c63ca5e9031741b0ee51e2254f92d.tar.gz rust-e928a8f4a05c63ca5e9031741b0ee51e2254f92d.zip | |
Rollup merge of #138432 - weihanglo:lld, r=lqd
fix: remove the check of lld not supporting @response-file In LLVM v9, lld has supported `@response-file.` LLVM v9 was released on 2019-09-19. The check was added back to 2018-03-14 (1.26.0) via 04442af18bf0. It has been more than five years, and we ship our own lld regardlessly. This should be happily removed. See also: * <https://github.com/llvm/llvm-project/commit/bb12396f9175eaf4586d8e5c76441977d97ccf93> * <https://reviews.llvm.org/D63024>
| -rw-r--r-- | compiler/rustc_codegen_ssa/src/back/command.rs | 7 |
1 files changed, 0 insertions, 7 deletions
diff --git a/compiler/rustc_codegen_ssa/src/back/command.rs b/compiler/rustc_codegen_ssa/src/back/command.rs index 63023fdba20..383d0579e52 100644 --- a/compiler/rustc_codegen_ssa/src/back/command.rs +++ b/compiler/rustc_codegen_ssa/src/back/command.rs @@ -143,13 +143,6 @@ impl Command { return false; } - // Right now LLD doesn't support the `@` syntax of passing an argument - // through files, so regardless of the platform we try to go to the OS - // on this one. - if let Program::Lld(..) = self.program { - return false; - } - // Ok so on Windows to spawn a process is 32,768 characters in its // command line [1]. Unfortunately we don't actually have access to that // as it's calculated just before spawning. Instead we perform a |