[PATCH] D85311: [analyzer][tests] Understand when diagnostics change between builds

Valeriy Savchenko via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Aug 5 07:40:00 PDT 2020


vsavchenko created this revision.
vsavchenko added reviewers: NoQ, dcoughlin, xazax.hun, Szelethus.
Herald added subscribers: cfe-commits, steakhal, ASDenysPetrov, Charusso, dkrupp, donat.nagy, mikhail.ramalho, a.sidorin, rnkovacs, szepet, baloghadamsoftware.
Herald added a project: clang.
vsavchenko requested review of this revision.

Before the patch `SATest compare`, produced quite obscure results
when something about the diagnostic have changed (i.e. its description
or the name of the corresponding checker) because it was simply two
lists of warnings, ADDED and REMOVED.  It was up to the developer
to match those warnings, understand that they are essentially the
same, and figure out what caused the difference.

This patch introduces another category of results: MODIFIED.
It tries to match new warnings against the old ones and prints out
clues on what is different between two builds.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D85311

Files:
  clang/utils/analyzer/CmpRuns.py

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D85311.283235.patch
Type: text/x-patch
Size: 12135 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20200805/17372768/attachment-0001.bin>


More information about the cfe-commits mailing list