[PATCH] D81413: [lld-macho] improve handling of -platform_version

[Jez Ng via Phabricator via llvm-commits]

int3 added inline comments.


================
Comment at: lld/test/MachO/platform-version.s:10
+# RUN:        -platform_version \
+# RUN:    | FileCheck --check-prefix=FAIL_MISSING %s
+# RUN: not lld -flavor darwinnew -Z -o %t %t.o 2>&1 \
----------------
nit: it's more common to use hyphens instead of underscores for check prefixes, i.e. `FAIL-MISSING`

Also, I was thinking it might make sense to split this test into two parts, with all the `FAIL-` checks going under `invalid/`, and the passing ones remaining here. Once we support `LC_BUILD_VERSION`, we'll probably want to expand the test for passing platform strings to include checks that the `LC_BUILD_VERSION` command contains the expected output.


================
Comment at: lld/test/MachO/platform-version.s:41
+# RUN: lld -flavor darwinnew -Z -o %t %t.o 2>&1 \
+# RUN:        -platform_version ios-sim 1.2.3 5.6.7
+# RUN: lld -flavor darwinnew -Z -o %t %t.o 2>&1 \
----------------
even though `man ld` indicates that `ios-sim` is a valid platform string, I couldn't get it to work on ld64 :/ it seems like `ios-simulator` is expected instead. (I tried that string out after finding it in rG25ce33a6e).


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D81413/new/

https://reviews.llvm.org/D81413