devnexen added a comment. In https://reviews.llvm.org/D42645#996668, @NoQ wrote:
> All right, so the code looks good now, but in order to commit this, we also > need tests. We've got those automatic tests of ours in `test/Analysis/` that > feed small code snippets into the analyzer and verify that it does (or > doesn't) emit warnings in certain cases, so you'd need to prove that your > checker works (as intended, ideally) by writing some of those > (http://clang-analyzer.llvm.org/checker_dev_manual.html#testing). Updated with a little new unit test. Repository: rC Clang https://reviews.llvm.org/D42645 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
