[PATCH] D20277: [clang-tidy] UnnecessaryValueParamCheck - suggest std::move() if non-const value parameter can be moved.

[Felix Berger via cfe-commits]

flx added a comment.

In http://reviews.llvm.org/D20277#436717, @Prazek wrote:

> Cool check! Did you think about sugesting std::move for rvalue references if they are used once?


Thanks! I'm not sure this fits with what a user would expect from a check named "unnecessary-value-param" since in this case the parameter is not passed by value. Would a separate check make sense for this?


http://reviews.llvm.org/D20277