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

Reply via email to