diff options
author | Douglas Gregor <dgregor@apple.com> | 2011-12-31 04:05:44 +0000 |
---|---|---|
committer | Douglas Gregor <dgregor@apple.com> | 2011-12-31 04:05:44 +0000 |
commit | 1fb5c3a63a88e4758b196f1bda64eecce9b29d3f (patch) | |
tree | 0da4770b525e3b6a77a4b54bd828262d2bda7dfa /clang/lib/Frontend/FrontendActions.cpp | |
parent | 84a5dfdf727df09432aaa40ac72aa94e500326e4 (diff) | |
download | bcm5719-llvm-1fb5c3a63a88e4758b196f1bda64eecce9b29d3f.tar.gz bcm5719-llvm-1fb5c3a63a88e4758b196f1bda64eecce9b29d3f.zip |
Implement support for module requirements, which indicate the language
features needed for a particular module to be available. This allows
mixed-language modules, where certain headers only work under some
language variants (e.g., in C++, std.tuple might only be available in
C++11 mode).
llvm-svn: 147387
Diffstat (limited to 'clang/lib/Frontend/FrontendActions.cpp')
-rw-r--r-- | clang/lib/Frontend/FrontendActions.cpp | 16 |
1 files changed, 15 insertions, 1 deletions
diff --git a/clang/lib/Frontend/FrontendActions.cpp b/clang/lib/Frontend/FrontendActions.cpp index a97a32ba067..dd98fcf3eb1 100644 --- a/clang/lib/Frontend/FrontendActions.cpp +++ b/clang/lib/Frontend/FrontendActions.cpp @@ -136,6 +136,10 @@ ASTConsumer *GenerateModuleAction::CreateASTConsumer(CompilerInstance &CI, static void collectModuleHeaderIncludes(const LangOptions &LangOpts, clang::Module *Module, llvm::SmallString<256> &Includes) { + // Don't collect any headers for unavailable modules. + if (!Module->isAvailable()) + return; + // Add includes for each of these headers. for (unsigned I = 0, N = Module->Headers.size(); I != N; ++I) { if (LangOpts.ObjC1) @@ -222,7 +226,17 @@ bool GenerateModuleAction::BeginSourceFileAction(CompilerInstance &CI, return false; } - + + // Check whether we can build this module at all. + StringRef Feature; + if (!Module->isAvailable(CI.getLangOpts(), Feature)) { + CI.getDiagnostics().Report(diag::err_module_unavailable) + << Module->getFullModuleName() + << Feature; + + return false; + } + // Do we have an umbrella header for this module? const FileEntry *UmbrellaHeader = Module->getUmbrellaHeader(); |