[cfe-dev] Clang Static Analyzer: False Positive Suppression Support

[p23 power via cfe-dev]

Hi,

Suppress using comments (or pragma):
>
Suppress using hashes
>

At Sony I have had a number of customer requests for this feature, i.e. the
analyzer to not output suppressed warnings. My preference is to feed in the
hashes to clang, the reason for this is three fold. (1) From an
investigation our customers do not want to add suppression's as comments.in
the source code.  (2) Different teams may want to handle suppression's
differently where many teams write/use the same code base (3) A developer
may want to suppress everything that is not theirs (only interested in new
warnings that they have introduced) whilst the overnight build may want no
suppressions (or limited suppressions).  Using the comment/pragma method
doesn't allow for configurability in different workflows.

Currently we provide tools that use the hashes stored in a simple json file
(and suppress in the report viewer). I was thinking along the lines of
passing this json file back into clang.  A json file containing a simple
list of hashes is relatively source control friendly, though it is one more
file to checkin (i.e. one file to store all hashes per project - obviously
developers could decide to do one file per source file too).

I understand that some developers/teams may prefer the comment or pragma
option.  Would anybody object to support both methods?  And are you open to
using a json file as the input?

Regards,
Phillip
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-dev/attachments/20160818/dc0a2e92/attachment.html>