[PATCH] D141569: [clang-tidy] Implement CppCoreGuideline F.18

[Chris Cotter via Phabricator via cfe-commits]

ccotter marked an inline comment as done.
ccotter added a comment.

In D141569#4048359 <https://reviews.llvm.org/D141569#4048359>, @njames93 wrote:

> What happens with code like this
>
>   void foo(bar&& B) {
>     std::move(B);
>   }

https://github.com/isocpp/CppCoreGuidelines/issues/2026#issuecomment-1402454584 pointed out there's another guideline, ES.56, which a "useless" `move` such as this example would violate. I imagine this should be a different clang-tidy check though?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D141569/new/

https://reviews.llvm.org/D141569