[PATCH] D42645: New simple Checker for mmap calls

David CARLIER via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Feb 2 17:42:53 PST 2018


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





More information about the cfe-commits mailing list