diff options
Diffstat (limited to 'src/link.zig')
| -rw-r--r-- | src/link.zig | 24 |
1 files changed, 23 insertions, 1 deletions
diff --git a/src/link.zig b/src/link.zig index 064c178d66..e8a9f3d730 100644 --- a/src/link.zig +++ b/src/link.zig @@ -238,7 +238,29 @@ pub const File = struct { pub fn makeExecutable(base: *File) !void { switch (base.tag) { - .coff, .elf, .macho => if (base.file) |f| { + .macho => if (base.file) |f| { + if (base.intermediary_basename != null) { + // The file we have open is not the final file that we want to + // make executable, so we don't have to close it. + return; + } + if (comptime std.Target.current.isDarwin() and std.Target.current.cpu.arch == .aarch64) { + if (base.options.target.cpu.arch != .aarch64) return; // If we're not targeting aarch64, nothing to do. + // XNU starting with Big Sur running on arm64 is caching inodes of running binaries. + // Any change to the binary will effectively invalidate the kernel's cache + // resulting in a SIGKILL on each subsequent run. Since when doing incremental + // linking we're modifying a binary in-place, this will end up with the kernel + // killing it on every subsequent run. To circumvent it, we will copy the file + // into a new inode, remove the original file, and rename the copy to match + // the original file. This is super messy, but there doesn't seem any other + // way to please the XNU. + const emit = base.options.emit orelse return; + try emit.directory.handle.copyFile(emit.sub_path, emit.directory.handle, emit.sub_path, .{}); + } + f.close(); + base.file = null; + }, + .coff, .elf => if (base.file) |f| { if (base.intermediary_basename != null) { // The file we have open is not the final file that we want to // make executable, so we don't have to close it. |