alexfh accepted this revision. alexfh added a comment. This revision is now accepted and ready to land.
This test doesn't fail without your fix. ================ Comment at: clang-tidy/modernize/PassByValueCheck.cpp:155 @@ -155,1 +154,3 @@ + .bind("Initializer")), + unless(isCopyConstructor())) .bind("Ctor"), ---------------- This submatcher is much cheaper, so it should go first to make the whole matcher more efficient. http://reviews.llvm.org/D19194 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits