[Lldb-commits] [PATCH] D61498: 01/06: Merge GetCompileUnit + GetCompileUnitContainingDIEOffset
Jan Kratochvil via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Fri May 3 06:15:52 PDT 2019
jankratochvil created this revision.
jankratochvil added reviewers: clayborg, labath, JDevlieghere.
jankratochvil added a project: LLDB.
Herald added a subscriber: abidh.
These two methods are very similar and various refactorizations need to modify both similar ways. Merge them as a template.
One could also just remove `GetCompileUnit` and make `GetCompileUnitContainingDIEOffset` to also accept offset of the CU itself (currently it accepts only DIE offsets after the CU header). But that would be less safe regarding some internal sanity checking.
The whole patchset available for testing as: `git clone -b debugtypes git://git.jankratochvil.net/lldb`
Repository:
rLLDB LLDB
https://reviews.llvm.org/D61498
Files:
lldb/source/Plugins/SymbolFile/DWARF/DWARFDebugInfo.cpp
lldb/source/Plugins/SymbolFile/DWARF/DWARFDebugInfo.h
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D61498.197977.patch
Type: text/x-patch
Size: 4359 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20190503/d7d1f369/attachment.bin>
More information about the lldb-commits
mailing list