[PATCH] D61364: [libcxx] [test] Fix incorrect allocator::construct assertions in associative container tests

[Eric Fiselier via Phabricator via cfe-commits]

EricWF added inline comments.


================
Comment at: test/std/containers/map_allocator_requirement_test_templates.h:236
     const ValueTp v(42, 1);
-    cc->expect<const ValueTp&>();
+    cc->expect<const ValueTp&&>();
     It ret = c.insert(c.end(), std::move(v));
----------------
I really think the current behavior libc++ has here is correct.




CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D61364/new/

https://reviews.llvm.org/D61364



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits