Skip to content

Commit d21bff8

Browse files
authored
Unrolled build for rust-lang#138432
Rollup merge of rust-lang#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 04442af. It has been more than five years, and we ship our own lld regardlessly. This should be happily removed. See also: * <llvm/llvm-project@bb12396> * <https://reviews.llvm.org/D63024>
2 parents 523c507 + c8a6fcc commit d21bff8

File tree

1 file changed

+0
-7
lines changed

1 file changed

+0
-7
lines changed

compiler/rustc_codegen_ssa/src/back/command.rs

-7
Original file line numberDiff line numberDiff line change
@@ -143,13 +143,6 @@ impl Command {
143143
return false;
144144
}
145145

146-
// Right now LLD doesn't support the `@` syntax of passing an argument
147-
// through files, so regardless of the platform we try to go to the OS
148-
// on this one.
149-
if let Program::Lld(..) = self.program {
150-
return false;
151-
}
152-
153146
// Ok so on Windows to spawn a process is 32,768 characters in its
154147
// command line [1]. Unfortunately we don't actually have access to that
155148
// as it's calculated just before spawning. Instead we perform a

0 commit comments

Comments
 (0)