malcolm.parsons added inline comments.
================ Comment at: test/clang-tidy/performance-unnecessary-value-param.cpp:276 +// Virtual method overrides of dependent types cannot be recognized unless they +// are marked as override or final. Test that check is not triggered on methods +// marked with override or final. ---------------- flx wrote: > malcolm.parsons wrote: > > Do we want to warn about methods in templated classes at all? > I think we should still trigger on methods where we can determine that the > parameter type is expensive to copy. Note that this is not the case when the > type is dependent. We still have issues when the overriding method is not marked override or final. Is there another way to handle this? https://reviews.llvm.org/D27248 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
