Quuxplusone requested changes to this revision. Quuxplusone added inline comments. This revision now requires changes to proceed.
================ Comment at: clang-tools-extra/test/clang-tidy/checkers/performance-move-const-arg.cpp:75 + return std::move(x4); + // CHECK-MESSAGES: :[[@LINE-1]]:10: warning: passing result of std::move() as a const reference argument; no move will actually happen + // CHECK-FIXES: return x4; ---------------- You're right that //some// diagnostic should be given here, but //this// is not the right diagnostic. The issue is not that the `std::move` is non-functional, but rather that it's superfluous; a move //will// happen, with or without the `std::move`. (And the `std::move` prevents move elision, so removing it may actually improve codegen.) ================ Comment at: clang-tools-extra/test/clang-tidy/checkers/performance-move-const-arg.cpp:261 + showInt(std::move(a)); + // CHECK-MESSAGES: :[[@LINE-1]]:11: warning: std::move of the variable 'a' of the trivially-copyable type 'int' has no effect; remove std::move() + return std::move(a); ---------------- This warning is incorrect, as indicated in your PR summary. Isn't the point of this PR mainly to remove this incorrect warning? CHANGES SINCE LAST ACTION https://reviews.llvm.org/D107450/new/ https://reviews.llvm.org/D107450 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
