FreeChainXenon/binutils-gdb
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Move the frame data to the BFD when possible

Browse source 
Now that comp_unit and the remaining frame data are all independent of
the objfile, it can all be stored on the BFD and shared across
inferiors.

As with other code doing this same thing, care must be taken to not
share the data when the objfile requires relocations.  So, two keys
are used: one for the BFD and one for the objfile, and
gdb_bfd_requires_relocations is used to differentiate between the two
cases.

gdb/ChangeLog
2020-02-12  Tom Tromey  <tom@tromey.com>

	* dwarf2/frame.c (dwarf2_frame_bfd_data): New global.
	(dwarf2_frame_objfile_data): Add comment.
	(find_comp_unit, set_comp_unit): New functions.
	(dwarf2_frame_find_fde): Use find_comp_unit.
	(dwarf2_build_frame_info): Use set_comp_unit.
This commit is contained in:
Tom Tromey 2020-02-12 15:45:08 -07:00
parent 2198230456
commit 3d4560f707
2 changed files with 41 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
gdb/ChangeLog
View file

@ -1,3 +1,11 @@
2020-02-12 Tom Tromey <tom@tromey.com>
* dwarf2/frame.c (dwarf2_frame_bfd_data): New global.
(dwarf2_frame_objfile_data): Add comment.
(find_comp_unit, set_comp_unit): New functions.
(dwarf2_frame_find_fde): Use find_comp_unit.
(dwarf2_build_frame_info): Use set_comp_unit.
2020-02-12 Tom Tromey <tom@tromey.com> 2020-02-12 Tom Tromey <tom@tromey.com>
* dwarf2/frame.c (struct comp_unit) <objfile>: Remove. * dwarf2/frame.c (struct comp_unit) <objfile>: Remove.

37
gdb/dwarf2/frame.c
View file

@ -1480,8 +1480,14 @@ dwarf2_frame_cfa (struct frame_info *this_frame)
return get_frame_base (this_frame); return get_frame_base (this_frame);
} }
static const struct objfile_key<comp_unit> dwarf2_frame_objfile_data; /* We store the frame data on the BFD. This is only done if it is
independent of the address space and so can be shared. */
static const struct bfd_key<comp_unit> dwarf2_frame_bfd_data;
/* If any BFD sections require relocations (note; really should be if
any debug info requires relocations), then we store the frame data
on the objfile instead, and do not share it. */
const struct objfile_key<comp_unit> dwarf2_frame_objfile_data;
/* Pointer encoding helper functions. */ /* Pointer encoding helper functions. */
@ -1635,6 +1641,29 @@ bsearch_fde_cmp (const dwarf2_fde *fde, CORE_ADDR seek_pc)
return 1; return 1;
} }
/* Find an existing comp_unit for an objfile, if any. */
static comp_unit *
find_comp_unit (struct objfile *objfile)
{
bfd *abfd = objfile->obfd;
if (gdb_bfd_requires_relocations (abfd))
return dwarf2_frame_bfd_data.get (abfd);
return dwarf2_frame_objfile_data.get (objfile);
}
/* Store the comp_unit on OBJFILE, or the corresponding BFD, as
appropriate. */
static void
set_comp_unit (struct objfile *objfile, struct comp_unit *unit)
{
bfd *abfd = objfile->obfd;
if (gdb_bfd_requires_relocations (abfd))
return dwarf2_frame_bfd_data.set (abfd, unit);
return dwarf2_frame_objfile_data.set (objfile, unit);
}
/* Find the FDE for *PC. Return a pointer to the FDE, and store the /* Find the FDE for *PC. Return a pointer to the FDE, and store the
initial location associated with it into *PC. */ initial location associated with it into *PC. */
@ -1646,11 +1675,11 @@ dwarf2_frame_find_fde (CORE_ADDR *pc, CORE_ADDR *out_offset)
CORE_ADDR offset; CORE_ADDR offset;
CORE_ADDR seek_pc; CORE_ADDR seek_pc;
comp_unit *unit = dwarf2_frame_objfile_data.get (objfile); comp_unit *unit = find_comp_unit (objfile);
if (unit == NULL) if (unit == NULL)
{ {
dwarf2_build_frame_info (objfile); dwarf2_build_frame_info (objfile);
unit = dwarf2_frame_objfile_data.get (objfile); unit = find_comp_unit (objfile);
} }
gdb_assert (unit != NULL); gdb_assert (unit != NULL);
@ -2262,7 +2291,7 @@ dwarf2_build_frame_info (struct objfile *objfile)
} }
unit->fde_table.shrink_to_fit (); unit->fde_table.shrink_to_fit ();
dwarf2_frame_objfile_data.set (objfile, unit.release ()); set_comp_unit (objfile, unit.release ());
} }
/* Handle 'maintenance show dwarf unwinders'. */ /* Handle 'maintenance show dwarf unwinders'. */