Author: ericwf Date: Fri May 26 21:46:17 2017 New Revision: 304054 URL: http://llvm.org/viewvc/llvm-project?rev=304054&view=rev Log: [coroutines] Support "coroutines" feature in module map requires clause
Summary: In order for libc++ to add `<experimental/coroutine>` to its module map, there has to be a feature that can be used to detect if coroutines support is enabled in Clang. Reviewers: rsmith Reviewed By: rsmith Subscribers: cfe-commits Differential Revision: https://reviews.llvm.org/D33538 Added: cfe/trunk/test/Modules/Inputs/DependsOnModule.framework/Headers/coroutines.h cfe/trunk/test/Modules/Inputs/DependsOnModule.framework/Headers/not_coroutines.h cfe/trunk/test/Modules/requires-coroutines.mm Modified: cfe/trunk/docs/Modules.rst cfe/trunk/lib/Basic/Module.cpp cfe/trunk/test/Modules/Inputs/DependsOnModule.framework/module.map Modified: cfe/trunk/docs/Modules.rst URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/docs/Modules.rst?rev=304054&r1=304053&r2=304054&view=diff ============================================================================== --- cfe/trunk/docs/Modules.rst (original) +++ cfe/trunk/docs/Modules.rst Fri May 26 21:46:17 2017 @@ -413,6 +413,9 @@ altivec blocks The "blocks" language feature is available. +coroutines + Support for the coroutines TS is available. + cplusplus C++ support is available. Modified: cfe/trunk/lib/Basic/Module.cpp URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Basic/Module.cpp?rev=304054&r1=304053&r2=304054&view=diff ============================================================================== --- cfe/trunk/lib/Basic/Module.cpp (original) +++ cfe/trunk/lib/Basic/Module.cpp Fri May 26 21:46:17 2017 @@ -64,6 +64,7 @@ static bool hasFeature(StringRef Feature bool HasFeature = llvm::StringSwitch<bool>(Feature) .Case("altivec", LangOpts.AltiVec) .Case("blocks", LangOpts.Blocks) + .Case("coroutines", LangOpts.CoroutinesTS) .Case("cplusplus", LangOpts.CPlusPlus) .Case("cplusplus11", LangOpts.CPlusPlus11) .Case("freestanding", LangOpts.Freestanding) Added: cfe/trunk/test/Modules/Inputs/DependsOnModule.framework/Headers/coroutines.h URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Modules/Inputs/DependsOnModule.framework/Headers/coroutines.h?rev=304054&view=auto ============================================================================== --- cfe/trunk/test/Modules/Inputs/DependsOnModule.framework/Headers/coroutines.h (added) +++ cfe/trunk/test/Modules/Inputs/DependsOnModule.framework/Headers/coroutines.h Fri May 26 21:46:17 2017 @@ -0,0 +1,3 @@ +#ifndef __cpp_coroutines +#error coroutines must be enabled +#endif Added: cfe/trunk/test/Modules/Inputs/DependsOnModule.framework/Headers/not_coroutines.h URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Modules/Inputs/DependsOnModule.framework/Headers/not_coroutines.h?rev=304054&view=auto ============================================================================== --- cfe/trunk/test/Modules/Inputs/DependsOnModule.framework/Headers/not_coroutines.h (added) +++ cfe/trunk/test/Modules/Inputs/DependsOnModule.framework/Headers/not_coroutines.h Fri May 26 21:46:17 2017 @@ -0,0 +1,3 @@ +#ifdef __cpp_coroutines +#error coroutines must NOT be enabled +#endif Modified: cfe/trunk/test/Modules/Inputs/DependsOnModule.framework/module.map URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Modules/Inputs/DependsOnModule.framework/module.map?rev=304054&r1=304053&r2=304054&view=diff ============================================================================== --- cfe/trunk/test/Modules/Inputs/DependsOnModule.framework/module.map (original) +++ cfe/trunk/test/Modules/Inputs/DependsOnModule.framework/module.map Fri May 26 21:46:17 2017 @@ -22,7 +22,14 @@ framework module DependsOnModule { explicit module CustomReq2 { requires custom_req2 } - + explicit module Coroutines { + requires coroutines + header "coroutines.h" + } + explicit module NotCoroutines { + requires !coroutines + header "not_coroutines.h" + } explicit framework module SubFramework { umbrella header "SubFramework.h" Added: cfe/trunk/test/Modules/requires-coroutines.mm URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Modules/requires-coroutines.mm?rev=304054&view=auto ============================================================================== --- cfe/trunk/test/Modules/requires-coroutines.mm (added) +++ cfe/trunk/test/Modules/requires-coroutines.mm Fri May 26 21:46:17 2017 @@ -0,0 +1,12 @@ +// RUN: rm -rf %t +// RUN: %clang_cc1 -Wauto-import -fmodules-cache-path=%t -fmodules -fimplicit-module-maps -F %S/Inputs %s -verify +// RUN: %clang_cc1 -Wauto-import -fmodules-cache-path=%t -fmodules -fimplicit-module-maps -F %S/Inputs %s -verify -fcoroutines-ts -DCOROUTINES + + +#ifdef COROUTINES +@import DependsOnModule.Coroutines; +@import DependsOnModule.NotCoroutines; // expected-error {{module 'DependsOnModule.NotCoroutines' is incompatible with feature 'coroutines'}} +#else +@import DependsOnModule.NotCoroutines; +@import DependsOnModule.Coroutines; // expected-error {{module 'DependsOnModule.Coroutines' requires feature 'coroutines'}} +#endif _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
