JDevlieghere added inline comments.
================ Comment at: lldb/source/Plugins/ObjectFile/ELF/ObjectFileELF.cpp:382 ObjectFile *ObjectFileELF::CreateMemoryInstance( const lldb::ModuleSP &module_sp, DataBufferSP &data_sp, const lldb::ProcessSP &process_sp, lldb::addr_t header_addr) { ---------------- JDevlieghere wrote: > labath wrote: > > I am assuming this will always point to a writable kind of a data buffer. > > Could we change the prototype to reflect that? > Are you okay with making that a separate patch? https://reviews.llvm.org/D123073 CHANGES SINCE LAST ACTION https://reviews.llvm.org/D122856/new/ https://reviews.llvm.org/D122856 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits