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
Updated with a little new unit test.
cfe-commits mailing list