[PATCH] D53982: Output "rule" information in SARIF
Aaron Ballman via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Nov 5 07:36:41 PST 2018
aaron.ballman added inline comments.
Comment at: lib/StaticAnalyzer/Core/SarifDiagnostics.cpp:237-242
+#define CHECKER(FULLNAME, CLASS, CXXFILE, HELPTEXT, GROUPINDEX, HIDDEN) \
+ .Case(FULLNAME, HELPTEXT)
> Szelethus wrote:
> > Hmmm, this won't handle checkers loaded from plugins.
> I don't immediately know the solution is to this, but when you invoke clang with `-cc1 -analyzer-checker-help`, plugin checkers are also displayed, and [[ https://github.com/llvm-mirror/clang/blob/master/lib/StaticAnalyzer/Frontend/CheckerRegistration.cpp#L150 | this is line that magically does it ]].
> Maybe store the plugins in `AnalyzerOptions`, and move `ClangCheckerRegistry` to `include/clang/StaticAnalyzer/Frontend`?
Oof, this is a good point. Thank you for bringing it up! I'll look into this as soon as I have the chance.
Do we have any tests that use plugins (preferably on Windows) so that I have a way to test this functionality out?
More information about the cfe-commits