[PATCH] D18637: Extract key to avoid preemptive mallocs in insert/emplace in associative containers

Eric Fiselier via cfe-commits cfe-commits at lists.llvm.org
Wed Apr 13 09:50:45 PDT 2016


EricWF added inline comments.

================
Comment at: test/std/containers/associative/set/insert_and_emplace_allocator_requirements.pass.cpp:29
@@ +28,3 @@
+template <class Arg>
+void PrintInfo(int line, Arg&& arg)
+{
----------------
mclow.lists wrote:
> In general, we want tests to be silent on success.
I'll see what I can do. The reason these tests are so noisy is because almost all of the assertions are in "container_test_types.h". Meaning that when the tests fail the diagnostic points into the header with no information about what code actually triggered the assertion.  Hence why these tests manually print what they are testing.

I'll try and come up with something better.


http://reviews.llvm.org/D18637





More information about the cfe-commits mailing list