[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
Thu Dec 13 14:50:20 PST 2018
arphaman marked an inline comment as done.
arphaman added inline comments.
================
Comment at: include/clang/Driver/DarwinSDKInfo.h:36
+/// SDK has no SDKSettings.json, or a valid \c DarwinSDKInfo otherwise.
+Expected<Optional<DarwinSDKInfo>> parseDarwinSDKInfo(llvm::vfs::FileSystem &VFS,
+ StringRef SDKRootPath);
----------------
steven_wu wrote:
> Isn't parseSDKSettings enough? And it can just return Optional<VersionTuple>?
We will support other fields besides `VersionTuple` in the SDKSettings, so that's why we have a structure.
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