clayborg requested changes to this revision.
clayborg added a comment.
This revision now requires changes to proceed.
See inlined comment. Quick fix and this will be good to go.
================
Comment at: source/Plugins/ObjectFile/ELF/ObjectFileELF.cpp:409-410
if (!data_sp) {
- data_sp =
- DataBufferLLVM::CreateSliceFromPath(file->GetPath(), length,
file_offset, true);
+ data_sp = DataBufferLLVM::CreateSliceFromPath(file->GetPath(), length,
+ file_offset);
if (!data_sp)
----------------
We should put the DataBufferLLVM::CreateSliceFromPath() call into a new static
function in ObjectFile:
```
static DataBufferSP ObjectFile::MapFileData(FileSpec file, uint64_t Size,
uint64_t Offset);
```
Then when we need to make fixes, we change just one place instead of all of
them. We should add this new function and switch all ObjectFile subclasses over
to using it.
https://reviews.llvm.org/D40079
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
