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

Richard Smith - zygoloid via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Sep 14 16:52:25 PDT 2018


rsmith added inline comments.


================
Comment at: docs/Modules.rst:476-477
+
+*platform-environment*
+  A platform-environment variant (e.g. ``linux-gnueabi``, ``windows-msvc``) is available.
 
----------------
What is the reason to allow these to be combined into a single feature name rather than treating them as two distinct features? (Eg, `requires linux, gnueabi` rather than `requires linux-gnueabi`)?


https://reviews.llvm.org/D51910





More information about the cfe-commits mailing list