[PATCH] D51910: [Modules] Add platform feature to requires clause

Adrian Prantl via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Sep 14 16:51:32 PDT 2018


aprantl added inline comments.


================
Comment at: lib/Basic/Module.cpp:101
+  // variant (2), the simulator is hardcoded as part of the platform name. Both
+  // forms above should match "iossimulator" and "ios-simulator" features.
+  if (Target.getTriple().isOSDarwin() && PlatformEnv.endswith("simulator"))
----------------
Ah.. it looks like you are fully aware of this :-)


https://reviews.llvm.org/D51910





More information about the cfe-commits mailing list