[PATCH 3/3] libdw: save startp/endp boundaries in Dwarf_CU
Mark Wielaard
mjw at redhat.com
Thu Dec 11 12:06:25 UTC 2014
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 <jistone at 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
More information about the elfutils-devel
mailing list