[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