[Lldb-commits] [PATCH] D61908: DWARF: Add ability to reference debug info coming from multiple sections

2019-05-16 Thread Pavel Labath via Phabricator via lldb-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL360872: DWARF: Add ability to reference debug info coming from multiple sections (authored by labath, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Changed prior

[Lldb-commits] [PATCH] D61908: DWARF: Add ability to reference debug info coming from multiple sections

2019-05-15 Thread Pavel Labath via Phabricator via lldb-commits
labath updated this revision to Diff 199562. labath marked 4 inline comments as done. labath added a comment. - use enum class for the section type - put DIERef as a member of HashedNameToDie::DIEInfo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D61908/new/

[Lldb-commits] [PATCH] D61908: DWARF: Add ability to reference debug info coming from multiple sections

2019-05-15 Thread Pavel Labath via Phabricator via lldb-commits
labath marked 10 inline comments as done. labath added inline comments. Comment at: source/Plugins/SymbolFile/DWARF/DIERef.cpp:18 if (form_value.IsValid()) { const DWARFUnit *dwarf_cu = form_value.GetCompileUnit(); if (dwarf_cu) { clayborg wrote: >

[Lldb-commits] [PATCH] D61908: DWARF: Add ability to reference debug info coming from multiple sections

2019-05-14 Thread Greg Clayton via Phabricator via lldb-commits
clayborg added a comment. See inline comments. Nice patch overall though, real close. Comment at: source/Plugins/SymbolFile/DWARF/DIERef.cpp:18 if (form_value.IsValid()) { const DWARFUnit *dwarf_cu = form_value.GetCompileUnit(); if (dwarf_cu) { We