[clang-tools-extra] [clang-tidy] Add check performance-lost-std-move (PR #139525)

[Vasiliy Kulikov via cfe-commits]

segoon wrote:

> What's the differences between this check and `modernize-pass-by-value`? Although the names are different, they are essentially checking similar things.

Sorry, as you didn't comment a specific line of code, I cannot reply directly to you. The checks are about the same std::move, but lost-std-move finds usages of should-be-moved, while pass-by-value finds constructors that can be transformed from T& to T+std::move (AFAICS). pass-by-value doesn't search for "last usage should be move'd".

https://github.com/llvm/llvm-project/pull/139525