[PATCH] D31975: [Analyzer] Iterator Checkers

Artem Dergachev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Apr 19 22:59:50 PDT 2017


NoQ added a comment.

Hmm.

1. Could you do renaming the checker file in a separate patch, so that we saw an actual diff, not a whole greenish file, here on phabricator?
2. The invalidated iterator checker looks relatively small (a single check and a few rounds of iterator invalidations), would it be hard to split it up into a separate patch? That'd leave us with just one checker added.


https://reviews.llvm.org/D31975





More information about the cfe-commits mailing list