On Wed, 2014-12-10 at 18:48 -0800, Josh Stone wrote: > Rather than looking up section data every time, we can simply save the > range of each CU as pointers in Dwarf_CU. > > 2014-12-10 Josh Stone <jist...@redhat.com> > + * libdwP.h (Dwarf_CU): Add startp and endp boundaries. > + * libdw_findcu.c (__libdw_intern_next_unit): Set startp and endp. > + * dwarf_child.c (dwarf_child): Use cu->endp. > + * dwarf_cuoffset.c (dwarf_cuoffset): Use cu->startp. > + * dwarf_dieoffset.c (dwarf_dieoffset): Use cu->startp. > + * dwarf_siblingof.c (dwarf_siblingof): Use both.
This looks good, thanks. (Nitpick, blank line again.) It also gave me an idea how to better handle bounds checking in the dwarf_form functions when we have "fake" attributes. Cheers, Mark
