diff options
-rw-r--r-- | include/clang/Sema/ParseAST.h | 2 | ||||
-rw-r--r-- | lib/CodeGen/ModuleBuilder.cpp | 19 | ||||
-rw-r--r-- | lib/Sema/ParseAST.cpp | 5 |
3 files changed, 15 insertions, 11 deletions
diff --git a/include/clang/Sema/ParseAST.h b/include/clang/Sema/ParseAST.h index 61ce30d9f4..9f6690a0bd 100644 --- a/include/clang/Sema/ParseAST.h +++ b/include/clang/Sema/ParseAST.h @@ -21,7 +21,7 @@ namespace clang { /// ParseAST - Parse the entire file specified, notifying the ASTConsumer as /// the file is parsed. This takes ownership of the ASTConsumer and /// ultimately deletes it. - void ParseAST(Preprocessor &pp, ASTConsumer *C, bool PrintStats = false); + void ParseAST(Preprocessor &pp, ASTConsumer *C, bool PrintStats = false, bool DeleteConsumer = true); } // end namespace clang #endif diff --git a/lib/CodeGen/ModuleBuilder.cpp b/lib/CodeGen/ModuleBuilder.cpp index 7329ed1d90..6ce4e6894d 100644 --- a/lib/CodeGen/ModuleBuilder.cpp +++ b/lib/CodeGen/ModuleBuilder.cpp @@ -49,13 +49,7 @@ namespace { virtual ~CodeGeneratorImpl() {} - virtual llvm::Module* ReleaseModule() { - if (Diags.hasErrorOccurred()) - return 0; - - if (Builder) - Builder->Release(); - + virtual llvm::Module* ReleaseModule() { return M.take(); } @@ -134,7 +128,16 @@ namespace { virtual void HandleTagDeclDefinition(TagDecl *D) { Builder->UpdateCompletedType(D); } - + + virtual void HandleTranslationUnit(TranslationUnit& TU) { + if (Diags.hasErrorOccurred()) { + M.reset(); + return; + } + + if (Builder) + Builder->Release(); + }; }; } diff --git a/lib/Sema/ParseAST.cpp b/lib/Sema/ParseAST.cpp index 94e0185f03..6b9b8d5804 100644 --- a/lib/Sema/ParseAST.cpp +++ b/lib/Sema/ParseAST.cpp @@ -27,7 +27,7 @@ using namespace clang; /// ParseAST - Parse the entire file specified, notifying the ASTConsumer as /// the file is parsed. This takes ownership of the ASTConsumer and /// ultimately deletes it. -void clang::ParseAST(Preprocessor &PP, ASTConsumer *Consumer, bool PrintStats) { +void clang::ParseAST(Preprocessor &PP, ASTConsumer *Consumer, bool PrintStats, bool DeleteConsumer) { // Collect global stats on Decls/Stmts (until we have a module streamer). if (PrintStats) { Decl::CollectingStats(true); @@ -78,5 +78,6 @@ void clang::ParseAST(Preprocessor &PP, ASTConsumer *Consumer, bool PrintStats) { Stmt::CollectingStats(false); } - delete Consumer; + if (DeleteConsumer) + delete Consumer; } |