[PATCH] D55673: [darwin] parse the SDK settings from SDKSettings.json if it exists and pass in the -target-sdk-version to the compiler and backend

Alex Lorenz via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Dec 14 15:42:37 PST 2018


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


================
Comment at: lib/Driver/ToolChains/Darwin.cpp:2053
+    return None;
+  }
+  return *SDKInfoOrErr;
----------------
steven_wu wrote:
> We also has this InferredFromSDK when we infer deployment target, which can be used as a fallback method.
> 
> The result of parsing JSON should be available to InferredFromSDK in deployment target setting.
> 
> Bonus point is to set "-sdk_version" flag passing to ld64.
Done.
We're not planning to pass it down to the linker.


Repository:
  rC Clang

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

https://reviews.llvm.org/D55673





More information about the cfe-commits mailing list