Re: PATCH to dwarf2read.c:decode_locdesc

Daniel Jacobowitz
2002-01-11T22:48:26Z

On Fri, Jan 11, 2002 at 10:12:39AM +0000, Jason Merrill wrote:
> >>>>> "Elena" == Elena Zannoni <ezannoni@redhat.com> writes:
>
> > Sigh! There have been several attempts to get the location expression
> > support in gdb. They all stopped dead for one reason or another. There
> > are several WIP patches out there.
> > http://sources.redhat.com/ml/gdb-patches/2001-02/msg00276.html
> > http://sources.redhat.com/ml/gdb-patches/2001-06/msg00390.html
> > http://sources.redhat.com/ml/gdb-patches/2001-06/msg00502.html
> > http://sources.redhat.com/ml/gdb-patches/2001-07/msg00008.html
>
> > Hmm, DW_OP_dup wasn't handled in the above patches.
>
> Actually, none of these patches make any attempt to generalize location
> expression handling. The first just copies some organizational changes
> from gcc. The others are Dan Berlin's WIP dwarf2read rewrite, which
> doesn't touch the locexp code (though I think it does fix inter-CU
> references, which is needed before we can do any duplicate elimination on
> the compiler/linker side).

For what it's worth, I'm working over the rewrite patches and breaking
them down into chunks now. inter-CU references aren't tricky once we
have some cleanup in place.

--
Daniel Jacobowitz Carnegie Mellon University
MontaVista Software Debian GNU/Linux Developer