This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [patch] Discontiguous PSYMTABs [Re: [patch] Parse DW_AT_ranges into PSYMTABS (for childless CU, for vDSO32)]
- From: Daniel Jacobowitz <drow at false dot org>
- To: Jan Kratochvil <jan dot kratochvil at redhat dot com>, gdb-patches at sources dot redhat dot com, Jim Blandy <jimb at codesourcery dot com>, Vladimir Prus <ghost at cs dot msu dot su>
- Date: Sun, 16 Dec 2007 20:02:17 -0500
- Subject: Re: [patch] Discontiguous PSYMTABs [Re: [patch] Parse DW_AT_ranges into PSYMTABS (for childless CU, for vDSO32)]
- References: <20071009180246.GA26829@host0.dyn.jankratochvil.net> <20071009181701.GA30757@caradoc.them.org> <20071009185434.GA27810@host0.dyn.jankratochvil.net> <20071009185931.GA1220@caradoc.them.org> <20071124154339.GA14913@host0.dyn.jankratochvil.net> <E1Iy0Vd-0006lL-64@zigzag.lvk.cs.msu.su> <20071130111021.GA17867@host0.dyn.jankratochvil.net> <20071130145613.GA5941@caradoc.them.org> <20071209203138.GA23339@host0.dyn.jankratochvil.net> <20071217005641.GA14690@caradoc.them.org>
On Sun, Dec 16, 2007 at 07:56:41PM -0500, Daniel Jacobowitz wrote:
> On Sun, Dec 09, 2007 at 09:31:38PM +0100, Jan Kratochvil wrote:
> > Attaching the fix for discontiguous psymtabs based on the addrmap framework.
> >
> > This one is a conservative one - it tries to just fix it with minimal changes.
> > It is bidirectionally compatible:
> > * Producer (dwarf2read.c) still tries to set the bounds TEXTLOW and TEXTHIGH.
> > * Consumer (symtab.c) deals with both set and unset PSYMTABS_ADDRMAP.
>
> This looks OK to me. Thanks for doing it!
Oops, maybe not. I've just thought of another case where we might
have trouble. What if one file contains DW_AT_ranges and another file
only contains stabs?
If we trust the addrmap, will we still find the file with stabs? We
may need to fall through.
I'm sure there are some old versions of GCC which generated
DW_AT_high_pc / DW_AT_low_pc when they should have generated
DW_AT_ranges. Hopefully we do not need to cater to that. However,
stabs has no way to represent range information.
--
Daniel Jacobowitz
CodeSourcery