[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
FileSpec minidump_file(filename.c_str(), false);
It looks like this variable is not necessary anymore.
More information about the lldb-commits