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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to