1
Fork 0

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 04442af18b.
It has been more than five years, and we ship our own lld regardlessly.
This should be happily removed.

See also:

* <bb12396f91>
* <https://reviews.llvm.org/D63024>
This commit is contained in:
Jacob Pratt 2025-03-14 01:37:33 -04:00 committed by GitHub
commit e928a8f4a0
No known key found for this signature in database
GPG key ID: B5690EEEBB952194

View file

@ -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