[PATCH] D71579: [driver][darwin] Pass -platform_version flag to the linker instead of the -<platform>_version_min flag

[Steven Wu via Phabricator via cfe-commits]

steven_wu added inline comments.


================
Comment at: clang/lib/Driver/ToolChains/Darwin.cpp:2530
+      return "ios";
+    // FIXME: Add macCatalyst support here ("\"mac catalyst\"").
+    llvm_unreachable("macCatalyst isn't yet supported");
----------------
arphaman wrote:
> steven_wu wrote:
> > Why not support that in this commit? I don't see problem returning a string with space inside.
> The rest of macCatalyst support is not yet upstreamed (specifically the new `Environment` value)
Doesn't this mean you hit llvm_unreachable when you link catalyst? And this is the default behavior?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D71579