Revert "[coroutines] Support "coroutines" feature in module map requires clause"
authorEric Fiselier <eric@efcs.ca>
Sat, 27 May 2017 03:04:51 +0000 (03:04 +0000)
committerEric Fiselier <eric@efcs.ca>
Sat, 27 May 2017 03:04:51 +0000 (03:04 +0000)
This reverts commit r304054.

llvm-svn: 304057

clang/docs/Modules.rst
clang/lib/Basic/Module.cpp
clang/test/Modules/Inputs/DependsOnModule.framework/Headers/coroutines.h [deleted file]
clang/test/Modules/Inputs/DependsOnModule.framework/Headers/not_coroutines.h [deleted file]
clang/test/Modules/Inputs/DependsOnModule.framework/module.map
clang/test/Modules/requires-coroutines.mm [deleted file]

index b8841c0..2b1bde2 100644 (file)
@@ -413,9 +413,6 @@ altivec
 blocks
   The "blocks" language feature is available.
 
-coroutines
-  Support for the coroutines TS is available.
-
 cplusplus
   C++ support is available.
 
index ac3d7c5..a6fd931 100644 (file)
@@ -64,7 +64,6 @@ static bool hasFeature(StringRef Feature, const LangOptions &LangOpts,
   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)
diff --git a/clang/test/Modules/Inputs/DependsOnModule.framework/Headers/coroutines.h b/clang/test/Modules/Inputs/DependsOnModule.framework/Headers/coroutines.h
deleted file mode 100644 (file)
index 85281f5..0000000
+++ /dev/null
@@ -1,3 +0,0 @@
-#ifndef __cpp_coroutines
-#error coroutines must be enabled
-#endif
diff --git a/clang/test/Modules/Inputs/DependsOnModule.framework/Headers/not_coroutines.h b/clang/test/Modules/Inputs/DependsOnModule.framework/Headers/not_coroutines.h
deleted file mode 100644 (file)
index 9312b9a..0000000
+++ /dev/null
@@ -1,3 +0,0 @@
-#ifdef __cpp_coroutines
-#error coroutines must NOT be enabled
-#endif
index 4d468f2..b623085 100644 (file)
@@ -22,14 +22,7 @@ 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"
 
diff --git a/clang/test/Modules/requires-coroutines.mm b/clang/test/Modules/requires-coroutines.mm
deleted file mode 100644 (file)
index d3519cd..0000000
+++ /dev/null
@@ -1,12 +0,0 @@
-// 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