about summary refs log tree commit diff
diff options
context:
space:
mode:
authorWeihang Lo <me@weihanglo.tw>2025-03-12 18:36:21 -0400
committerWeihang Lo <me@weihanglo.tw>2025-03-12 18:36:21 -0400
commitc8a6fcc3c838af8152fda0a22a8946914aaa5770 (patch)
tree4980ce200b4611370159780b913297c51506f3fc
parent249cb84316401daf040832cdbb8a45e0f5ab6af8 (diff)
downloadrust-c8a6fcc3c838af8152fda0a22a8946914aaa5770.tar.gz
rust-c8a6fcc3c838af8152fda0a22a8946914aaa5770.zip
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.
And 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.rs7
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