aaron.ballman added inline comments.
================ Comment at: lib/StaticAnalyzer/Core/SarifDiagnostics.cpp:237-242 +#define GET_CHECKERS +#define CHECKER(FULLNAME, CLASS, CXXFILE, HELPTEXT, GROUPINDEX, HIDDEN) \ + .Case(FULLNAME, HELPTEXT) +#include "clang/StaticAnalyzer/Checkers/Checkers.inc" +#undef CHECKER +#undef GET_CHECKERS ---------------- Szelethus wrote: > 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? https://reviews.llvm.org/D53982 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits