Febbe added a comment. In D137205#3923643 <https://reviews.llvm.org/D137205#3923643>, @Skylion007 wrote:
> The main false positive I also keep seeing is in pybind11 where it suggests > call an std::move() for an implicit conversion, but the target of the > implicit conversion does not have an rvalue. (In this case, we have a > py::object which inherits from py::handle, and is just py::handle with > ownership semantics. This allows implicit conversion to a reference at > anytime, and therefore std::move has no effect here except to complicate the > code a bit. Can you post preferably a minimal reproducible example? Alternatively, you could write me again the exact position of the false positive. Including the implicit conversion operator. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D137205/new/ https://reviews.llvm.org/D137205 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
