[PATCH] Include checker name in Static Analyzer PLIST output
Anna Zaks
zaks.anna at gmail.com
Tue Jan 6 10:23:51 PST 2015
I don't believe the checker name should be used for bug identification. The checker names are implementation detail. The bug message/name and category are better for this. If we think that we might be changing the names of categories, we might come up with some kind of a stable ID.
For example, this patch, which is currently in review (http://reviews.llvm.org/D6178), will move the implementation of a set of warnings from one place/checker to another.
I'd suggest to only use the bug message and it's location. (We should already do that in the CmpRuns script.)
What do you think?
http://reviews.llvm.org/D6841
EMAIL PREFERENCES
http://reviews.llvm.org/settings/panel/emailpreferences/
More information about the cfe-commits
mailing list