[lld/mac] Warn that writing zippered outputs isn't implemented
authorNico Weber <thakis@chromium.org>
Wed, 20 Apr 2022 16:36:16 +0000 (12:36 -0400)
committerNico Weber <thakis@chromium.org>
Thu, 21 Apr 2022 16:05:56 +0000 (12:05 -0400)
commit889847922dc6b9247f7f9189cf06e46fa5591049
tree247c4deaf7aaf08b84612eed107d90142337a4d1
parentad46aaede6e4d5a6951fc9827da994d3fbe1af44
[lld/mac] Warn that writing zippered outputs isn't implemented

A "zippered" dylib contains several LC_BUILD_VERSION load commands, usually
one each for "normal" macOS and one for macCatalyst.

These are usually created by passing something like

   -shared -target arm64-apple-macos -darwin-target-variant arm64-apple-ios13.1-macabi

to clang, which turns it into

    -platform_version macos 12.0.0 12.3 -platform_version "mac catalyst" 14.0.0 15.4

for the linker.

ld64.lld can read these files fine, but it can't write them.  Before this
change, it would just silently use the last -platform_version flag and ignore
the rest.

This change adds a warning that writing zippered dylibs isn't implemented yet
instead.

Sadly, parts of ld64.lld's test suite relied on the previous
"silently use last flag" semantics for its test suite: `%lld` always expanded
to `ld64.lld -platform_version macos 10.15 11.0` and tests that wanted a
different value passed a 2nd `-platform_version` flag later on. But this now
produces a warning if the platform passed to `-platform_version` is not `macos`.

There weren't very many cases of this, so move these to use `%no-arg-lld` and
manually pass `-arch`.

Differential Revision: https://reviews.llvm.org/D124106
lld/MachO/Driver.cpp
lld/MachO/InputFiles.cpp
lld/test/MachO/invalid/incompatible-arch.s
lld/test/MachO/invalid/incompatible-target-tapi.test
lld/test/MachO/lc-build-version.s
lld/test/MachO/objc-uses-custom-personality.s
lld/test/MachO/platform-version.s
lld/test/MachO/tapi-link-by-arch.s
lld/test/MachO/zippered.yaml