DwarfExtensions (Was: Recognize .debug_macro/DW_AT_GNU_macros)

2012-07-23 Thread Mark Wielaard
On Wed, Jun 27, 2012 at 10:17:52AM +0200, Mark Wielaard wrote: I removed the reference from the dwarf.h file, but don't agree the constants alone are enough to know where one can find real information (at least I often spend a lot of time trying to hunt down information on obscure DWARF

Re: Recognize .debug_macro/DW_AT_GNU_macros

2012-07-19 Thread Mark Wielaard
On Wed, 2012-07-11 at 19:14 +0200, Mark Wielaard wrote: Here is a patch for readelf raw .debug_macro reading. The output is like .debug_macinfo, but the format was different enough to just add a new section parsing function. I merged and pushed this part now. The new .debug_macro format has

Re: Recognize .debug_macro/DW_AT_GNU_macros

2012-07-11 Thread Mark Wielaard
Here is a patch for readelf raw .debug_macro reading. The output is like .debug_macinfo, but the format was different enough to just add a new section parsing function. The vendor extension opcode table code isn't really tested since there are no examples out there yet. But I had some idea for

Re: Recognize .debug_macro/DW_AT_GNU_macros

2012-06-27 Thread Mark Wielaard
On Tue, 2012-06-26 at 18:14 -0700, Roland McGrath wrote: dwarf.h should get all the new constants I added the new constants in a followup commit. and it doesn't need comments with references. The DW_*_GNU_* names are enough to know what to look up. I removed the reference from the dwarf.h

Recognize .debug_macro/DW_AT_GNU_macros

2012-06-26 Thread Mark Wielaard
Hi, This adds minimal low-level support for .debug_macro/DW_AT_GNU_macros as proposed here http://www.dwarfstd.org/ShowIssue.php?issue=110722.1 and implemented in gcc 4.7 (-g3). It is just enough to not barf on it when we encounter the new attribute and to sanity check the attribute form offsets.

Re: Recognize .debug_macro/DW_AT_GNU_macros

2012-06-26 Thread Roland McGrath
dwarf.h should get all the new constants, and it doesn't need comments with references. The DW_*_GNU_* names are enough to know what to look up. If we can support the new format entirely with the existing libdw API, then I agree that is the right thing to do. Thanks, Roland