This is an archive of the discontinued LLVM Phabricator instance.

[DebugInfo] Move all DWARF headers to the public include directory.
ClosedPublic

Authored by friss on Dec 16 2014, 2:32 PM.

Details

Summary

dsymutil needs access to DWARF specific inforamtion, the small DIContext
wrapper isn't sufficient. Other DWARF consumers might want to use it too
(I'm looking at you lldb).

I have discussed that briefly with David at the dev meeting, it seemed
OK.

Diff Detail

Repository
rL LLVM

Event Timeline

friss updated this revision to Diff 17365.Dec 16 2014, 2:32 PM
friss retitled this revision from to [DebugInfo] Move all DWARF headers to the public include directory..
friss added reviewers: echristo, dblaikie, samsonov.
friss updated this object.
friss added a subscriber: Unknown Object (MLST).
dblaikie accepted this revision.Dec 16 2014, 2:37 PM
dblaikie edited edge metadata.

Fine by me - happy to discuss it with Alexey & Eric post-commit if there's anything more to consider.

This revision is now accepted and ready to land.Dec 16 2014, 2:37 PM
echristo edited edge metadata.Dec 16 2014, 3:31 PM

Nah. Go ahead.

This revision was automatically updated to reflect the committed changes.
llvm/trunk/lib/DebugInfo/DWARFTypeUnit.h