arphaman added inline comments.
================
Comment at: lib/Driver/ToolChains/Darwin.cpp:1457-1465
if (iOSVersion && (getTriple().getArch() == llvm::Triple::x86 ||
getTriple().getArch() == llvm::Triple::x86_64))
Platform = IPhoneOSSimulator;
if (TvOSVersion && (getTriple().getArch() == llvm::Triple::x86 ||
getTriple().getArch() == llvm::Triple::x86_64))
Platform = TvOSSimulator;
if (WatchOSVersion && (getTriple().getArch() == llvm::Triple::x86 ||
----------------
bob.wilson wrote:
> These checks should set the simulator target as well.
This would break the `__APPLE_EMBEDDED_SIMULATOR__` macro, as that has to be
specified only when is used `-m(iphone|tv|watch)simulator-version-min`. Or
should we still set `__APPLE_EMBEDDED_SIMULATOR__` in the driver and allow
`-simulator` even without `-m(iphone|tv|watch)simulator-version-min`?
Repository:
rC Clang
https://reviews.llvm.org/D40682
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
