Skip to content

Commit a3261a2

Browse files
committed
Revert "Link LLVM dynamically on aarch64-apple-darwin"
This reverts commit e7c268f.
1 parent 41c932c commit a3261a2

3 files changed

Lines changed: 3 additions & 18 deletions

File tree

compiler/rustc_llvm/build.rs

Lines changed: 0 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -402,16 +402,6 @@ fn main() {
402402
continue;
403403
}
404404

405-
// On apple-darwin, llvm-config reports the versioned shared library name such as LLVM-22-..., but
406-
// the distributed toolchain ships libLLVM.dylib. Normalize the link name here.
407-
let name =
408-
if target.contains("apple-darwin") && llvm_kind == "dylib" && name.starts_with("LLVM-")
409-
{
410-
"LLVM"
411-
} else {
412-
name
413-
};
414-
415405
let kind = if name.starts_with("LLVM") {
416406
llvm_kind
417407
} else if is_static {

compiler/rustc_metadata/src/native_libs.rs

Lines changed: 3 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -62,15 +62,12 @@ pub fn walk_native_lib_search_dirs<R>(
6262
f(&sess.target_tlib_path.dir.join("self-contained"), false)?;
6363
}
6464

65-
let has_shared_llvm_apple_darwin =
66-
sess.target.is_like_darwin && sess.target_tlib_path.dir.join("libLLVM.dylib").exists();
67-
6865
// Toolchains for some targets may ship `libunwind.a`, but place it into the main sysroot
6966
// library directory instead of the self-contained directories.
7067
// Sanitizer libraries have the same issue and are also linked by name on Apple targets.
7168
// The targets here should be in sync with `copy_third_party_objects` in bootstrap.
72-
// On Apple targets, shared LLVM is linked by name, so when `libLLVM.dylib` is
73-
// present in the target libdir, add that directory to the linker search path.
69+
// Finally there is shared LLVM library, which unlike compiler libraries, is linked by the name,
70+
// therefore requiring the search path for the linker.
7471
// FIXME: implement `-Clink-self-contained=+/-unwind,+/-sanitizers`, move the shipped libunwind
7572
// and sanitizers to self-contained directory, and stop adding this search path.
7673
// FIXME: On AIX this also has the side-effect of making the list of library search paths
@@ -80,8 +77,7 @@ pub fn walk_native_lib_search_dirs<R>(
8077
|| sess.target.os == Os::Linux
8178
|| sess.target.os == Os::Fuchsia
8279
|| sess.target.is_like_aix
83-
|| sess.target.is_like_darwin
84-
&& (!sess.sanitizers().is_empty() || has_shared_llvm_apple_darwin)
80+
|| sess.target.is_like_darwin && !sess.sanitizers().is_empty()
8581
|| sess.target.os == Os::Windows
8682
&& sess.target.env == Env::Gnu
8783
&& sess.target.cfg_abi == CfgAbi::Llvm

src/ci/github-actions/jobs.yml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -518,7 +518,6 @@ auto:
518518
--enable-sanitizers
519519
--enable-profiler
520520
--set rust.jemalloc
521-
--set llvm.link-shared=true
522521
--set rust.lto=thin
523522
--set rust.codegen-units=1
524523
# Aarch64 tooling only needs to support macOS 11.0 and up as nothing else

0 commit comments

Comments
 (0)