aaron.ballman accepted this revision. aaron.ballman added a comment. This revision is now accepted and ready to land.
LGTM! Please run your patch through clang-format before committing (just to fix the last bits of style and whitespace issues; no additional review required). Thank you for your hard work on this! As a cleanup once this goes in: we should add some user-facing documentation to docs\clang-tidy\checks for it. We can even take what's in ThrowByValueCatchByReferenceCheck.h and use that (then link to it from the header file instead of duplicating the words). http://reviews.llvm.org/D11328 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
