[PATCH] D26453: [clang-tidy] Remove duplicated check from move-constructor-init

[Felix Berger via cfe-commits]

flx added a comment.

Is the modernize-pass-by-value check configurable in a way to only trigger when copied constructor arguments are not moved?

I think our use case has been to have move-constructor-init check enabled to catch cases that can be optimized but not trigger on every constructor that uses const& instead of value + move, i.e. modernizing is not prescribed, but catching cases where the argument is already passed by value and making them faster is desirable.


https://reviews.llvm.org/D26453