[PATCH] D40682: [driver] Set the 'simulator' environment for Darwin when -m<os>simulator-version-min is used

[Bob Wilson via Phabricator via cfe-commits]

bob.wilson 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 ||
----------------
arphaman wrote:
> 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`?
The intention is that using a target triple with "simulator" in the environment should replace the use of -m*simulator-version-min.


Repository:
  rC Clang

https://reviews.llvm.org/D40682