llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT--> @llvm/pr-subscribers-clang-tidy Author: Jan Svoboda (jansvoboda11) <details> <summary>Changes</summary> --- Full diff: https://github.com/llvm/llvm-project/pull/141131.diff 2 Files Affected: - (modified) clang-tools-extra/clang-tidy/plugin/ClangTidyPlugin.cpp (+3-2) - (modified) clang/include/clang/Frontend/LogDiagnosticPrinter.h (-1) ``````````diff diff --git a/clang-tools-extra/clang-tidy/plugin/ClangTidyPlugin.cpp b/clang-tools-extra/clang-tidy/plugin/ClangTidyPlugin.cpp index 7911583db30e4..8c98ba7b9238a 100644 --- a/clang-tools-extra/clang-tidy/plugin/ClangTidyPlugin.cpp +++ b/clang-tools-extra/clang-tidy/plugin/ClangTidyPlugin.cpp @@ -40,9 +40,10 @@ class ClangTidyPluginAction : public PluginASTAction { // Create and set diagnostics engine auto *DiagConsumer = new ClangTidyDiagnosticConsumer(*Context, &Compiler.getDiagnostics()); + auto DiagOpts = std::make_unique<DiagnosticOptions>(); auto DiagEngine = std::make_unique<DiagnosticsEngine>( - new DiagnosticIDs, new DiagnosticOptions, DiagConsumer); - Context->setDiagnosticsEngine(DiagEngine.get()); + new DiagnosticIDs, *DiagOpts, DiagConsumer); + Context->setDiagnosticsEngine(std::move(DiagOpts), DiagEngine.get()); // Create the AST consumer. ClangTidyASTConsumerFactory Factory(*Context); diff --git a/clang/include/clang/Frontend/LogDiagnosticPrinter.h b/clang/include/clang/Frontend/LogDiagnosticPrinter.h index b43b0da13967a..9807dfa3aba1a 100644 --- a/clang/include/clang/Frontend/LogDiagnosticPrinter.h +++ b/clang/include/clang/Frontend/LogDiagnosticPrinter.h @@ -51,7 +51,6 @@ class LogDiagnosticPrinter : public DiagnosticConsumer { raw_ostream &OS; std::unique_ptr<raw_ostream> StreamOwner; const LangOptions *LangOpts; - DiagnosticOptions &DiagOpts; SourceLocation LastWarningLoc; FullSourceLoc LastLoc; `````````` </details> https://github.com/llvm/llvm-project/pull/141131 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
