[PATCH] D46902: [analyzer] Make plist-html multi-file.

Artem Dergachev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue May 15 13:19:21 PDT 2018


NoQ created this revision.
NoQ added reviewers: dcoughlin, xazax.hun, a.sidorin, george.karpenkov, szepet, rnkovacs.
Herald added subscribers: cfe-commits, baloghadamsoftware.

Plist diagnostics support multi-file reports since forever. HTML diagnostics support multi-file reports since https://reviews.llvm.org/D30406/https://reviews.llvm.org/rC309968.

plist-html is the diagnostic output mode that produces both html and plist files. It's mostly useful for testing the analyzer itself because plist output is helpful for comparing results produced by different versions of the analyzer via the `utils/analyzer/CmpRuns.py` and html output allows you to immediately have a look at the changed reports.

Previously plist-html output produced multi-file HTML reports but only single-file Plist reports.

Change plist-html output to produce multi-file Plist reports as well.

I don't think we should care about backwards compatibility here because not only the old mode made no sense but also it doesn't make sense to use plist-html in any user-facing UI anyway.


Repository:
  rC Clang

https://reviews.llvm.org/D46902

Files:
  lib/StaticAnalyzer/Frontend/AnalysisConsumer.cpp
  test/Analysis/diagnostics/plist-multi-file.c
  test/Analysis/diagnostics/plist-multi-file.h

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D46902.146902.patch
Type: text/x-patch
Size: 10371 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20180515/c1f145ba/attachment-0001.bin>


More information about the cfe-commits mailing list