Auto merge of #88250 - rusticstuff:macos-lld, r=nagisa
Make `-Z gcc-ld=lld` work for Apple targets `-Z gcc-ld=lld` was introduced in #85961. It does not work on Macos because lld needs be either named `ld64` or passed `-flavor darwin` as the first two arguments in order to select the Mach-O flavor. Rust invokes cc (=clang) on Macos for linking which calls `ld` as linker binary and not `ld64`, so just creating an `ld64` binary and modifying the search path with `-B` does not work. In order to solve this patch does: * Set the `lld_flavor` for all Apple-derived targets to `LldFlavor::Ld64`. As far as I can see this actually works towards fixing `-Xlinker=rust-lld` as all those targets use the Mach-O object format. * Copy/hardlink rust-lld to the gcc-ld subdirectory as ld64 next to ld. * If `-Z gcc-ld=lld` is used and the target lld flavor is Ld64 add `-fuse-ld=/path/to/ld64` to the linker invocation. Fixes #86945.
This commit is contained in:
commit
757a65bfdf
4 changed files with 41 additions and 15 deletions
|
@ -2482,13 +2482,31 @@ fn add_gcc_ld_path(cmd: &mut dyn Linker, sess: &Session, flavor: LinkerFlavor) {
|
||||||
if let LinkerFlavor::Gcc = flavor {
|
if let LinkerFlavor::Gcc = flavor {
|
||||||
match ld_impl {
|
match ld_impl {
|
||||||
LdImpl::Lld => {
|
LdImpl::Lld => {
|
||||||
|
if sess.target.lld_flavor == LldFlavor::Ld64 {
|
||||||
|
let tools_path =
|
||||||
|
sess.host_filesearch(PathKind::All).get_tools_search_paths(false);
|
||||||
|
let ld64_exe = tools_path
|
||||||
|
.into_iter()
|
||||||
|
.map(|p| p.join("gcc-ld"))
|
||||||
|
.map(|p| {
|
||||||
|
p.join(if sess.host.is_like_windows { "ld64.exe" } else { "ld64" })
|
||||||
|
})
|
||||||
|
.find(|p| p.exists())
|
||||||
|
.unwrap_or_else(|| sess.fatal("rust-lld (as ld64) not found"));
|
||||||
|
cmd.cmd().arg({
|
||||||
|
let mut arg = OsString::from("-fuse-ld=");
|
||||||
|
arg.push(ld64_exe);
|
||||||
|
arg
|
||||||
|
});
|
||||||
|
} else {
|
||||||
let tools_path =
|
let tools_path =
|
||||||
sess.host_filesearch(PathKind::All).get_tools_search_paths(false);
|
sess.host_filesearch(PathKind::All).get_tools_search_paths(false);
|
||||||
let lld_path = tools_path
|
let lld_path = tools_path
|
||||||
.into_iter()
|
.into_iter()
|
||||||
.map(|p| p.join("gcc-ld"))
|
.map(|p| p.join("gcc-ld"))
|
||||||
.find(|p| {
|
.find(|p| {
|
||||||
p.join(if sess.host.is_like_windows { "ld.exe" } else { "ld" }).exists()
|
p.join(if sess.host.is_like_windows { "ld.exe" } else { "ld" })
|
||||||
|
.exists()
|
||||||
})
|
})
|
||||||
.unwrap_or_else(|| sess.fatal("rust-lld (as ld) not found"));
|
.unwrap_or_else(|| sess.fatal("rust-lld (as ld) not found"));
|
||||||
cmd.cmd().arg({
|
cmd.cmd().arg({
|
||||||
|
@ -2498,6 +2516,7 @@ fn add_gcc_ld_path(cmd: &mut dyn Linker, sess: &Session, flavor: LinkerFlavor) {
|
||||||
});
|
});
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
}
|
||||||
} else {
|
} else {
|
||||||
sess.fatal("option `-Z gcc-ld` is used even though linker flavor is not gcc");
|
sess.fatal("option `-Z gcc-ld` is used even though linker flavor is not gcc");
|
||||||
}
|
}
|
||||||
|
|
|
@ -1,6 +1,6 @@
|
||||||
use std::env;
|
use std::env;
|
||||||
|
|
||||||
use crate::spec::{FramePointer, SplitDebuginfo, TargetOptions};
|
use crate::spec::{FramePointer, LldFlavor, SplitDebuginfo, TargetOptions};
|
||||||
|
|
||||||
pub fn opts(os: &str) -> TargetOptions {
|
pub fn opts(os: &str) -> TargetOptions {
|
||||||
// ELF TLS is only available in macOS 10.7+. If you try to compile for 10.6
|
// ELF TLS is only available in macOS 10.7+. If you try to compile for 10.6
|
||||||
|
@ -35,6 +35,7 @@ pub fn opts(os: &str) -> TargetOptions {
|
||||||
abi_return_struct_as_int: true,
|
abi_return_struct_as_int: true,
|
||||||
emit_debug_gdb_scripts: false,
|
emit_debug_gdb_scripts: false,
|
||||||
eh_frame_header: false,
|
eh_frame_header: false,
|
||||||
|
lld_flavor: LldFlavor::Ld64,
|
||||||
|
|
||||||
// The historical default for macOS targets is to run `dsymutil` which
|
// The historical default for macOS targets is to run `dsymutil` which
|
||||||
// generates a packed version of debuginfo split from the main file.
|
// generates a packed version of debuginfo split from the main file.
|
||||||
|
|
|
@ -1133,6 +1133,10 @@ impl Step for Assemble {
|
||||||
&lld_install.join("bin").join(&src_exe),
|
&lld_install.join("bin").join(&src_exe),
|
||||||
&gcc_ld_dir.join(exe("ld", target_compiler.host)),
|
&gcc_ld_dir.join(exe("ld", target_compiler.host)),
|
||||||
);
|
);
|
||||||
|
builder.copy(
|
||||||
|
&lld_install.join("bin").join(&src_exe),
|
||||||
|
&gcc_ld_dir.join(exe("ld64", target_compiler.host)),
|
||||||
|
);
|
||||||
}
|
}
|
||||||
|
|
||||||
// Similarly, copy `llvm-dwp` into libdir for Split DWARF. Only copy it when the LLVM
|
// Similarly, copy `llvm-dwp` into libdir for Split DWARF. Only copy it when the LLVM
|
||||||
|
|
|
@ -412,6 +412,8 @@ impl Step for Rustc {
|
||||||
let gcc_lld_dir = dst_dir.join("gcc-ld");
|
let gcc_lld_dir = dst_dir.join("gcc-ld");
|
||||||
t!(fs::create_dir(&gcc_lld_dir));
|
t!(fs::create_dir(&gcc_lld_dir));
|
||||||
builder.copy(&src_dir.join(&rust_lld), &gcc_lld_dir.join(exe("ld", compiler.host)));
|
builder.copy(&src_dir.join(&rust_lld), &gcc_lld_dir.join(exe("ld", compiler.host)));
|
||||||
|
builder
|
||||||
|
.copy(&src_dir.join(&rust_lld), &gcc_lld_dir.join(exe("ld64", compiler.host)));
|
||||||
}
|
}
|
||||||
|
|
||||||
// Copy over llvm-dwp if it's there
|
// Copy over llvm-dwp if it's there
|
||||||
|
|
Loading…
Add table
Add a link
Reference in a new issue