HomePhabricator

DWARF: Add support for type units+split dwarf combo

Description

DWARF: Add support for type units+split dwarf combo

Summary:
With the last round of refactors, supporting type units in dwo files
becomes almost trivial. This patch contains a couple of small fixes,
which taken as a whole make type units work in the split dwarf scenario
(both DWARF4 and DWARF5):

  • DWARFContext: make sure we actually read the debug_types.dwo section
  • DWARFUnit: set string offsets base on all units in the dwo file, not just the main CU
  • ManualDWARFIndex: index all units in the file
  • SymbolFileDWARFDwo: Search for the single compile unit in the file, as we can no longer assume it will be the first one

The last part makes it obvious that there is still some work to be done
here, namely that we do not support dwo files with multiple compile
units. That is something that should be easier after the DIERef
refactors, but it still requires more work.

Tests are added for the type units+split dwarf + dwarf4/5 scenarios, as
well as a test that checks we behave reasonably in the presence of dwo
files with multiple CUs.

Reviewers: clayborg, JDevlieghere, aprantl

Subscribers: arphaman, lldb-commits

Differential Revision: https://reviews.llvm.org/D63643

Details

Committed
labathJun 24 2019, 11:59 PM
Differential Revision
D63643: DWARF: Add support for type units+split dwarf combo
Branches
Unknown
Tags
Unknown