This patch helps add support for emitting the .debug_pubnames section to yaml2elf.
Known issues:
- Current implementation doesn't support emitting multiple sets of entries.
- Doesn't support DWARF64.
Differential D82296
[ObjectYAML][ELF] Add support for emitting the .debug_pubnames section. Higuoxing on Jun 22 2020, 6:27 AM. Authored by
Details This patch helps add support for emitting the .debug_pubnames section to yaml2elf. Known issues:
Diff Detail
Event TimelineComment Actions This patch LGTM. Don't spend hours of time working on supporting .pubnames/.pubtypes however, as they aren't widely used by my understanding. Still, for completeness, it doesn't hurt to support them for now. Comment Actions Yeah, I will not spend too much time on them. In the next patch, I will replace the InitialLength with Format and Length just as what we do in other sections. As for supporting multiple sets of entries, if nobody requests it, I will not work on it. Comment Actions I think if you end up with some spare time towards the end of the project, it doesn't hurt to "finish this off". Just not at the cost of more useful features. I do agree with the Length change though. Consistency is quite important.
|