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
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
