EricWF added a comment. Have you verified that we're not losing test coverage here? That is, are you sure we still have tests for the rvalue overloads in other test files?
================ Comment at: libcxx/test/std/containers/associative/multiset/insert_cv.pass.cpp:41 + const VT v3(3); + r = m.insert(3); + assert(r == prev(m.end())); ---------------- Should be `v3`? Repository: rCXX libc++ https://reviews.llvm.org/D44411 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits