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

Alex Lorenz via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 16 16:07:06 PST 2019


arphaman marked 2 inline comments as done.
arphaman added inline comments.


================
Comment at: clang/lib/Driver/ToolChains/Darwin.cpp:339
   // Add the deployment target.
-  MachOTC.addMinVersionArgs(Args, CmdArgs);
+  if (!Version[0] || Version[0] >= 520)
+    MachOTC.addPlatformVersionArgs(Args, CmdArgs);
----------------
steven_wu wrote:
> Why version '0' should go through the new path?
Version `0` indicates that `-mlinker-version=` wasn't specified. We want to use the new flag by default.


================
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");
----------------
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)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D71579





More information about the llvm-commits mailing list