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

Reply via email to