From b9574aceae0ca1d666797324f659ab17d2c8b857 Mon Sep 17 00:00:00 2001 From: David Blaikie Date: Thu, 17 Jul 2014 20:40:47 +0000 Subject: [PATCH] unique_ptr-ify ownership of ASTConsumers llvm-svn: 213308 --- .../clang-modernize/Core/Transform.cpp | 3 ++- clang-tools-extra/clang-tidy/ClangTidy.cpp | 27 +++++++++++----------- clang-tools-extra/clang-tidy/ClangTidy.h | 4 ++-- clang-tools-extra/modularize/Modularize.cpp | 8 +++---- .../module-map-checker/ModuleMapChecker.cpp | 7 +++--- clang-tools-extra/pp-trace/PPTrace.cpp | 7 +++--- .../unittests/clang-modernize/TransformTest.cpp | 4 ++-- 7 files changed, 32 insertions(+), 28 deletions(-) diff --git a/clang-tools-extra/clang-modernize/Core/Transform.cpp b/clang-tools-extra/clang-modernize/Core/Transform.cpp index 6bb1478..30caab6aa 100644 --- a/clang-tools-extra/clang-modernize/Core/Transform.cpp +++ b/clang-tools-extra/clang-modernize/Core/Transform.cpp @@ -49,7 +49,8 @@ private: FactoryAdaptor(MatchFinder &Finder, Transform &Owner) : Finder(Finder), Owner(Owner) {} - ASTConsumer *CreateASTConsumer(CompilerInstance &, StringRef) { + std::unique_ptr CreateASTConsumer(CompilerInstance &, + StringRef) { return Finder.newASTConsumer(); } diff --git a/clang-tools-extra/clang-tidy/ClangTidy.cpp b/clang-tools-extra/clang-tidy/ClangTidy.cpp index 98a5c618..3445fb5 100644 --- a/clang-tools-extra/clang-tidy/ClangTidy.cpp +++ b/clang-tools-extra/clang-tidy/ClangTidy.cpp @@ -179,10 +179,10 @@ private: class ClangTidyASTConsumer : public MultiplexConsumer { public: - ClangTidyASTConsumer(const SmallVectorImpl &Consumers, + ClangTidyASTConsumer(std::vector> Consumers, std::unique_ptr Finder, std::vector> Checks) - : MultiplexConsumer(Consumers), Finder(std::move(Finder)), + : MultiplexConsumer(std::move(Consumers)), Finder(std::move(Finder)), Checks(std::move(Checks)) {} private: @@ -203,8 +203,8 @@ ClangTidyASTConsumerFactory::ClangTidyASTConsumerFactory( } } - -clang::ASTConsumer *ClangTidyASTConsumerFactory::CreateASTConsumer( +std::unique_ptr +ClangTidyASTConsumerFactory::CreateASTConsumer( clang::CompilerInstance &Compiler, StringRef File) { // FIXME: Move this to a separate method, so that CreateASTConsumer doesn't // modify Compiler. @@ -224,7 +224,7 @@ clang::ASTConsumer *ClangTidyASTConsumerFactory::CreateASTConsumer( Check->registerPPCallbacks(Compiler); } - SmallVector Consumers; + std::vector> Consumers; if (!Checks.empty()) Consumers.push_back(Finder->newASTConsumer()); @@ -240,15 +240,16 @@ clang::ASTConsumer *ClangTidyASTConsumerFactory::CreateASTConsumer( AnalyzerOptions->AnalysisDiagOpt = PD_NONE; AnalyzerOptions->AnalyzeNestedBlocks = true; AnalyzerOptions->eagerlyAssumeBinOpBifurcation = true; - ento::AnalysisASTConsumer *AnalysisConsumer = ento::CreateAnalysisConsumer( - Compiler.getPreprocessor(), Compiler.getFrontendOpts().OutputFile, - AnalyzerOptions, Compiler.getFrontendOpts().Plugins); + std::unique_ptr AnalysisConsumer = + ento::CreateAnalysisConsumer( + Compiler.getPreprocessor(), Compiler.getFrontendOpts().OutputFile, + AnalyzerOptions, Compiler.getFrontendOpts().Plugins); AnalysisConsumer->AddDiagnosticConsumer( new AnalyzerDiagnosticConsumer(Context)); - Consumers.push_back(AnalysisConsumer); + Consumers.push_back(std::move(AnalysisConsumer)); } - return new ClangTidyASTConsumer(Consumers, std::move(Finder), - std::move(Checks)); + return llvm::make_unique( + std::move(Consumers), std::move(Finder), std::move(Checks)); } std::vector @@ -339,8 +340,8 @@ ClangTidyStats runClangTidy(ClangTidyOptionsProvider *OptionsProvider, class Action : public ASTFrontendAction { public: Action(ClangTidyASTConsumerFactory *Factory) : Factory(Factory) {} - ASTConsumer *CreateASTConsumer(CompilerInstance &Compiler, - StringRef File) override { + std::unique_ptr CreateASTConsumer(CompilerInstance &Compiler, + StringRef File) override { return Factory->CreateASTConsumer(Compiler, File); } diff --git a/clang-tools-extra/clang-tidy/ClangTidy.h b/clang-tools-extra/clang-tidy/ClangTidy.h index 8b181fc..cec52c0 100644 --- a/clang-tools-extra/clang-tidy/ClangTidy.h +++ b/clang-tools-extra/clang-tidy/ClangTidy.h @@ -99,8 +99,8 @@ public: ClangTidyASTConsumerFactory(ClangTidyContext &Context); /// \brief Returns an ASTConsumer that runs the specified clang-tidy checks. - clang::ASTConsumer *CreateASTConsumer(clang::CompilerInstance &Compiler, - StringRef File); + std::unique_ptr + CreateASTConsumer(clang::CompilerInstance &Compiler, StringRef File); /// \brief Get the list of enabled checks. std::vector getCheckNames(ChecksFilter &Filter); diff --git a/clang-tools-extra/modularize/Modularize.cpp b/clang-tools-extra/modularize/Modularize.cpp index db67754..730e62f 100644 --- a/clang-tools-extra/modularize/Modularize.cpp +++ b/clang-tools-extra/modularize/Modularize.cpp @@ -652,10 +652,10 @@ public: HadErrors(HadErrors) {} protected: - virtual clang::ASTConsumer *CreateASTConsumer(CompilerInstance &CI, - StringRef InFile) { - return new CollectEntitiesConsumer(Entities, PPTracker, - CI.getPreprocessor(), InFile, HadErrors); + std::unique_ptr + CreateASTConsumer(CompilerInstance &CI, StringRef InFile) override { + return llvm::make_unique( + Entities, PPTracker, CI.getPreprocessor(), InFile, HadErrors); } private: diff --git a/clang-tools-extra/module-map-checker/ModuleMapChecker.cpp b/clang-tools-extra/module-map-checker/ModuleMapChecker.cpp index d4c8b72..43f3cd45 100644 --- a/clang-tools-extra/module-map-checker/ModuleMapChecker.cpp +++ b/clang-tools-extra/module-map-checker/ModuleMapChecker.cpp @@ -178,9 +178,10 @@ public: ModuleMapCheckerAction(ModuleMapChecker &Checker) : Checker(Checker) {} protected: - virtual ASTConsumer *CreateASTConsumer(CompilerInstance &CI, - StringRef InFile) { - return new ModuleMapCheckerConsumer(Checker, CI.getPreprocessor()); + std::unique_ptr CreateASTConsumer(CompilerInstance &CI, + StringRef InFile) override { + return llvm::make_unique(Checker, + CI.getPreprocessor()); } private: diff --git a/clang-tools-extra/pp-trace/PPTrace.cpp b/clang-tools-extra/pp-trace/PPTrace.cpp index ed955fb..9dc1bbb 100644 --- a/clang-tools-extra/pp-trace/PPTrace.cpp +++ b/clang-tools-extra/pp-trace/PPTrace.cpp @@ -120,9 +120,10 @@ public: : Ignore(Ignore), CallbackCalls(CallbackCalls) {} protected: - virtual clang::ASTConsumer *CreateASTConsumer(CompilerInstance &CI, - StringRef InFile) { - return new PPTraceConsumer(Ignore, CallbackCalls, CI.getPreprocessor()); + std::unique_ptr + CreateASTConsumer(CompilerInstance &CI, StringRef InFile) override { + return llvm::make_unique(Ignore, CallbackCalls, + CI.getPreprocessor()); } private: diff --git a/clang-tools-extra/unittests/clang-modernize/TransformTest.cpp b/clang-tools-extra/unittests/clang-modernize/TransformTest.cpp index ae05f5a..6115efd 100644 --- a/clang-tools-extra/unittests/clang-modernize/TransformTest.cpp +++ b/clang-tools-extra/unittests/clang-modernize/TransformTest.cpp @@ -93,8 +93,8 @@ public: }; struct ConsumerFactory { - ASTConsumer *newASTConsumer() { - return new TimePassingASTConsumer(&Called); + std::unique_ptr newASTConsumer() { + return llvm::make_unique(&Called); } bool Called; }; -- 2.7.4