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

Felix Berger via cfe-commits cfe-commits at lists.llvm.org
Mon Nov 7 13:52:33 PST 2016


flx marked an inline comment as done.
flx added inline comments.


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


https://reviews.llvm.org/D26369





More information about the cfe-commits mailing list