[PATCH] D26369: [ClangTidy] Move incomplete type test into separate test file

Aaron Ballman via cfe-commits cfe-commits at lists.llvm.org
Mon Nov 7 13:49:23 PST 2016


aaron.ballman accepted this revision.
aaron.ballman added a comment.
This revision is now accepted and ready to land.

LGTM!



================
Comment at: test/clang-tidy/performance-unnecessary-value-param.cpp:257
-
-void PositiveNonConstDeclaration(ExpensiveToCopyType A);
-// CHECK-FIXES: void PositiveNonConstDeclaration(const ExpensiveToCopyType& A);
----------------
flx wrote:
> aaron.ballman wrote:
> > Why are these tests being removed?
> They should not be removed. Restored them.
Yay code review process! :-)


https://reviews.llvm.org/D26369





More information about the cfe-commits mailing list