[Lldb-commits] [PATCH] D30054: Delete DataBufferMemoryMap

Pavel Labath via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Wed Feb 22 06:50:57 PST 2017

labath added a comment.

I think you are still waiting to get the llvm changes sorted out, but this side of it looks fine to me (modulo a couple of nits).

Comment at: lldb/source/Plugins/ObjectFile/ELF/ObjectFileELF.cpp:414
+    data_sp = std::make_shared<DataBufferLLVM>(std::move(*Buffer));
+    data_offset = 0;
It looks like this make-a-MemoryBuffer-and-shove-it-in-a-DataBufferLLVM functionality could be handled by a utility function, as it is used in a number of places.

Comment at: lldb/unittests/Process/minidump/MinidumpParserTest.cpp:54
     llvm::sys::path::append(filename, minidump_filename);
     FileSpec minidump_file(filename.c_str(), false);
It looks like this variable is not necessary anymore.


More information about the lldb-commits mailing list