diff --git a/clang/include/clang/Driver/Options.td b/clang/include/clang/Driver/Options.td --- a/clang/include/clang/Driver/Options.td +++ b/clang/include/clang/Driver/Options.td @@ -3667,7 +3667,7 @@ def _all_warnings : Flag<["--"], "all-warnings">, Alias; def _analyzer_no_default_checks : Flag<["--"], "analyzer-no-default-checks">, Flags<[NoXarchOption]>; def _analyzer_output : JoinedOrSeparate<["--"], "analyzer-output">, Flags<[NoXarchOption]>, - HelpText<"Static analyzer report output format (html|plist|plist-multi-file|plist-html|sarif|text).">; + HelpText<"Static analyzer report output format (html|plist|plist-multi-file|plist-html|sarif|sarif-html|text).">; def _analyze : Flag<["--"], "analyze">, Flags<[NoXarchOption, CoreOption]>, HelpText<"Run the static analyzer">; def _assemble : Flag<["--"], "assemble">, Alias; diff --git a/clang/include/clang/StaticAnalyzer/Core/Analyses.def b/clang/include/clang/StaticAnalyzer/Core/Analyses.def --- a/clang/include/clang/StaticAnalyzer/Core/Analyses.def +++ b/clang/include/clang/StaticAnalyzer/Core/Analyses.def @@ -52,9 +52,14 @@ "Output analysis results using HTML wrapped with Plists", createPlistHTMLDiagnosticConsumer) -ANALYSIS_DIAGNOSTICS(SARIF, "sarif", "Output analysis results in a SARIF file", +ANALYSIS_DIAGNOSTICS(SARIF, "sarif", "Output analysis results using SARIF", createSarifDiagnosticConsumer) +ANALYSIS_DIAGNOSTICS(SARIF_HTML, "sarif-html", + "Output analysis results using both SARIF and HTML " + "output files", + createSarifHTMLDiagnosticConsumer) + ANALYSIS_DIAGNOSTICS(TEXT, "text", "Text output of analysis results to stderr", createTextPathDiagnosticConsumer) diff --git a/clang/lib/StaticAnalyzer/Core/HTMLDiagnostics.cpp b/clang/lib/StaticAnalyzer/Core/HTMLDiagnostics.cpp --- a/clang/lib/StaticAnalyzer/Core/HTMLDiagnostics.cpp +++ b/clang/lib/StaticAnalyzer/Core/HTMLDiagnostics.cpp @@ -176,6 +176,16 @@ CTU); } +void ento::createSarifHTMLDiagnosticConsumer( + PathDiagnosticConsumerOptions DiagOpts, PathDiagnosticConsumers &C, + const std::string &sarif_file, const Preprocessor &PP, + const cross_tu::CrossTranslationUnitContext &CTU) { + createHTMLDiagnosticConsumer(DiagOpts, C, std::string(llvm::sys::path::parent_path(sarif_file)), PP, CTU); + createSarifDiagnosticConsumer(DiagOpts, C, sarif_file, PP, CTU); + createTextMinimalPathDiagnosticConsumer(std::move(DiagOpts), C, sarif_file, PP, + CTU); +} + //===----------------------------------------------------------------------===// // Report processing. //===----------------------------------------------------------------------===//