• Mark Mentovai's avatar
    mac-arm64: Provide an arm64-native ninja-mac for builds in ¼ of the time · 4005bb74
    Mark Mentovai authored
    ninja-mac is expanded to become a universal (“fat”) executable. The
    existing x86_64 slice was taken from the x86_64 ninja-mac as it
    previously existed in this location, and a new arm64 slice was added.
    
    This is an aid to native development on arm64. In macOS, subprocesses
    inherit their parent’s “binpref,” meaning that children of a translated
    process will, by default, also be translated, if there’s a choice
    between native and translated. Where ninja-mac is x86_64-only and runs
    translated on arm64, all tools it invokes will also run translated, even
    if native versions are available. This has an impact on any part of the
    toolchain for which universal versions are available: in Chrome’s case,
    tools such as ld64 are affected. In Crashpad’s build, which uses the
    compiler provided by Xcode, it even affects the compiler, leading to
    significant differences in build time on arm64.
    
    mark@arm-and-hammer zsh% file $(which ninja-mac)
    …/ninja-mac: Mach-O 64-bit executable x86_64
    mark@arm-and-hammer zsh% time ninja -C out/release
    ninja: Entering directory `out/release'
    [1/1] Regenerating ninja files
    [511/511] LINK ./crashpad_minidump_test
    ninja -C out/release  1384.03s user 111.05s system 393% cpu 6:19.53 total
    
    mark@arm-and-hammer zsh% file $(which ninja-mac)
    …/ninja-mac: Mach-O universal binary with 2 architectures: [x86_64:Mach-O 64-bit executable x86_64] [arm64:Mach-O 64-bit executable arm64]
    …/ninja-mac (for architecture x86_64):	Mach-O 64-bit executable x86_64
    …/ninja-mac (for architecture arm64):	Mach-O 64-bit executable arm64
    mark@arm-and-hammer zsh% time ninja -C out/release
    ninja: Entering directory `out/release'
    [1/1] Regenerating ninja files
    [511/511] LINK ./crashpad_minidump_test
    ninja -C out/release  606.80s user 34.71s system 701% cpu 1:31.41 total
    
    This differs from the previous attempt, e64cd3ae
    (https://chromium-review.googlesource.com/c/2288417) in that the arm64
    version was built with Xcode 12.2b1 (12B5018i). The linker in this
    version of Xcode produces linker-signed output for arm64 by default. As
    of macOS 11.0db6 (20A5364e), all arm64 code must be signed. More
    information at
    https://developer.apple.com/documentation/macos-release-notes/macos-big-sur-11-universal-apps-beta-release-notes#Code-Signing.
    In the previous attempt, an older Xcode was used and the output was not
    signed at all. Because it did not run on systems upgraded to 11.0db6,
    this version of ninja-mac was backed out in a070dd18
    (https://chromium-review.googlesource.com/c/2393775).
    
    To create this ninja-mac:
    
    % git clone https://github.com/ninja-build/ninja.git
    % cd ninja
    % git checkout v1.8.2
    % mkdir ../build
    % cd ../build
    % LDFLAGS=-Wl,-s ../ninja/configure.py --bootstrap
    % lipo -create -output ninja-mac $(which ninja-mac) ninja
    
    An explanation of the LDFLAGS: e64cd3ae used “strip” on the output,
    matching the stripped x86_64 slice, but “strip” is not currently
    compatible with linker-signed executables. This is documented at
    https://developer.apple.com/documentation/xcode-release-notes/xcode-12_2-beta-release-notes#Apple-Clang-Compiler.
    I was unable to use the recommended workaround in this case. Rather than
    leave the output unstripped, I asked ld64 to strip symbols using its -s
    option, which it believes is obsolete, but in fact provides the intended
    behavior.
    
    The x86_64 slice is unsigned, as it was previously. The new arm64 slice
    is linker-signed with an ad-hoc signature. The x86_64 slice was built
    with the macOS 10.12 SDK and runs on Mac OS X 10.6 or later. The arm64
    slice was built with and has a minimum runtime requirement of macOS
    11.0. In practice, the difference doesn’t matter, as the 11.0 SDK is
    required to build for arm64, and no macOS version earlier than 11.0 will
    run on arm64.
    
    Bug: chromium:1103315
    Change-Id: I9f201d0d6f476e187acc0008681c3fd2c1a72aa0
    Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2427051Reviewed-by: 's avatarNico Weber <thakis@chromium.org>
    Commit-Queue: Mark Mentovai <mark@chromium.org>
    4005bb74