[Lldb-commits] [PATCH] D47508: [lldb-test] Add a testing harness for the JIT's IRMemoryMap

Pavel Labath via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Wed May 30 11:57:23 PDT 2018


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





More information about the lldb-commits mailing list