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 18, 2023
1 parent 4eb1b52 commit b7e98e1
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 b7e98e1

Please sign in to comment.