NoQ accepted this revision. NoQ added a comment. This revision is now accepted and ready to land.
Cool! I guess we still need to think how exactly do we want to resolve conflicts between dependencies. ================ Comment at: include/clang/StaticAnalyzer/Frontend/CheckerRegistry.h:152 + // FIXME: This *really* should be added to the frontend flag descriptions. /// Initializes a CheckerManager by calling the initialization functions for ---------------- Indeed! And, which is probably even more important, to `scan-build`'s help (assuming that the respective flags of `scan-build` do actually work in a similar manner). ================ Comment at: lib/StaticAnalyzer/Frontend/CheckerRegistry.cpp:143 + // Would it be better to name it '~experimental' or something else + // that's ASCIIbetically last? + llvm::sort(Checkers, checkerNameLT); ---------------- I learned a new word today :) :) Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D56989/new/ https://reviews.llvm.org/D56989 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
