[PATCH] D27248: [clang-tidy] Do not trigger unnecessary-value-param check on methods marked as final

[Felix Berger via Phabricator via cfe-commits]

flx 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.
----------------
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.


https://reviews.llvm.org/D27248