Skip to content

Commit

Permalink
Document that the macabi sanitizers are shared with darwin
Browse files Browse the repository at this point in the history
Do not rename and resign the darwin sanitizers a second time for
macabi.
  • Loading branch information
danakj committed Sep 11, 2023
1 parent dcbe9f7 commit 1365771
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions src/bootstrap/compile.rs
Original file line number Diff line number Diff line change
Expand Up @@ -570,13 +570,14 @@ fn copy_sanitizers(
let dst = libdir.join(&runtime.name);
builder.copy(&runtime.path, &dst);

// The `aarch64-apple-ios-macabi` and `x86_64-apple-ios-macabi` are also supported for
// sanitizers, but they share a sanitizer runtime with `${arch}-apple-darwin`, so we do
// not list them here to rename and sign the runtime library.
if target == "x86_64-apple-darwin"
|| target == "aarch64-apple-darwin"
|| target == "aarch64-apple-ios"
|| target == "aarch64-apple-ios-sim"
|| target == "aarch64-apple-ios-catalyst"
|| target == "x86_64-apple-ios"
|| target == "x86_64-apple-ios-catalyst"
{
// Update the library’s install name to reflect that it has been renamed.
apple_darwin_update_library_name(&dst, &format!("@rpath/{}", &runtime.name));
Expand Down

0 comments on commit 1365771

Please sign in to comment.