[PATCH] D15456: [PATCH] New diagnostic for non-idiomatic copy or move operations (v2)

Aaron Ballman via cfe-commits cfe-commits at lists.llvm.org
Fri Dec 11 10:15:36 PST 2015


aaron.ballman added a comment.

I should note that the majority of the changes in this patch are to deal with the diagnostic triggering on test cases. I tried to correct the test cases to be idiomatic where it appears the signature of the operation was not the purpose of the test. The other cases either got an expected-warning, or had the flag disabled.

I ran the diagnostic over several code bases, and the outcome was:

LLVM: 0 warnings
Qt: 7206 warnings (most of these are duplicate diagnostics because the signatures are in a header file)
rethinkdb: 34 warnings (same here)
cocos2d: 0 warnings
opencv: 0 warnings


http://reviews.llvm.org/D15456





More information about the cfe-commits mailing list