labath accepted this revision.
labath added a comment.
This revision is now accepted and ready to land.

Thank you for making the changes. This looks fine to me. The more testing, the 
better.



================
Comment at: tools/lldb-test/lldb-test.cpp:532
+  // Print the result of the allocation before checking its validity.
+  outs() << format("Malloc: address = 0x%x\n", Addr);
+
----------------
`%x` is not right here. Maybe use `formatv`?


https://reviews.llvm.org/D47508



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

Reply via email to