From b7e98e13cdbd6319fab38b6edc233e3d2925d567 Mon Sep 17 00:00:00 2001 From: danakj Date: Mon, 11 Sep 2023 10:33:50 -0400 Subject: [PATCH] Document that the macabi sanitizers are shared with darwin Do not rename and resign the darwin sanitizers a second time for macabi. --- src/bootstrap/compile.rs | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/src/bootstrap/compile.rs b/src/bootstrap/compile.rs index e02415153a84f..4f19ffa83dbe8 100644 --- a/src/bootstrap/compile.rs +++ b/src/bootstrap/compile.rs @@ -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));