NoQ added inline comments.
================ Comment at: clang/include/clang/StaticAnalyzer/Core/AnalyzerOptions.h:209 AnalysisConstraints AnalysisConstraintsOpt = RangeConstraintsModel; - AnalysisDiagClients AnalysisDiagOpt = PD_HTML; + AnalysisDiagClients AnalysisDiagOpt = PD_TEXT_MINIMAL; AnalysisPurgeMode AnalysisPurgeOpt = PurgeStmt; ---------------- This patch most likely changes almost nothing because the default output mode of `clang --analyze` has always been Plist: ```lang=c++ lib/Driver/ToolChains/Clang.cpp 2938: // Set the output format. The default is plist, for (lame) historical reasons. 2939- CmdArgs.push_back("-analyzer-output"); 2940- if (Arg *A = Args.getLastArg(options::OPT__analyzer_output)) 2941- CmdArgs.push_back(A->getValue()); 2942- else 2943- CmdArgs.push_back("plist"); ``` I'll double check on historical reasons; i suspect we should be able to transition out of it eventually because this is indeed the worst possible output mode for `--analyze`. That said, i wonder how/why do our tests currently work without flooding everything with html reports. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D76510/new/ https://reviews.llvm.org/D76510 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits