[PATCH] D112879: [MachO] Use error instead of fatal for missing -arch

[Jez Ng via Phabricator via llvm-commits]

int3 added inline comments.


================
Comment at: lld/test/MachO/color-diagnostics.test:9
 
-# COLOR: {{lld: .\[0;31merror: .\[0munknown argument '-xyz'}}
-# COLOR: {{lld: .\[0;31merror: .\[0mcannot open /nosuchfile}}
+# COLOR-UNKNOWN-ARGUMENT: {{lld: .\[0;31merror: .\[0munknown argument '-xyz'}}
+
----------------
do we really need to test this particular error? I was under the impression that this test was meant solely to test that the correct ANSI color codes were emitted, and the exact error message isn't so important


================
Comment at: lld/test/MachO/search-paths-darwin.test:6
 
-RUN: not ld64.lld -arch x86_64 -v -L%t1 -F%t2 2>&1 | FileCheck -DLDIR=%t1 -DFDIR=%t2 %s
+RUN: not ld64.lld -arch x86_64 -platform_version macos 10.5 11.0 -v -L%t1 -F%t2 2>&1 | FileCheck -DLDIR=%t1 -DFDIR=%t2 %s
 CHECK:      Library search paths:
----------------
hm, why can't we use `%lld` here?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D112879