espressif/binutils-gdb
1
0
Fork 0
mirror of https://github.com/espressif/binutils-gdb.git synced 2025-06-21 02:24:17 +08:00
Code Issues Packages Projects Releases Wiki Activity

pack partial_symtab for space

Browse Source 
This improves the packing of struct partial_symtab.  I noticed with
pahole that were were a couple of holes.  This consolidates the holes
without, I think, affecting readability -- it just moves the "user"
field a bit earlier in the struct.  This change saves a small amount
of memory.

2013-12-06  Tom Tromey  <tromey@redhat.com>

	* psympriv.h (struct partial_symtab) <user>: Move earlier.
This commit is contained in:
Tom Tromey
2013-11-25 09:34:49 -07:00
parent 2b69941d0d
commit e1b06ae220
2 changed files with 17 additions and 13 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
gdb/ChangeLog
View File

@ -1,3 +1,7 @@
2013-12-06 Tom Tromey <tromey@redhat.com>
* psympriv.h (struct partial_symtab) <user>: Move earlier.
2013-12-06 Tom Tromey <tromey@redhat.com> 2013-12-06 Tom Tromey <tromey@redhat.com>
* cli/cli-cmds.c (edit_command): Use paddress, not hex_string. * cli/cli-cmds.c (edit_command): Use paddress, not hex_string.

26
gdb/psympriv.h
View File

@ -111,19 +111,6 @@ struct partial_symtab
CORE_ADDR textlow; CORE_ADDR textlow;
CORE_ADDR texthigh; CORE_ADDR texthigh;
/* Array of pointers to all of the partial_symtab's which this one
depends on. Since this array can only be set to previous or
the current (?) psymtab, this dependency tree is guaranteed not
to have any loops. "depends on" means that symbols must be read
for the dependencies before being read for this psymtab; this is
for type references in stabs, where if foo.c includes foo.h, declarations
in foo.h may use type numbers defined in foo.c. For other debugging
formats there may be no need to use dependencies. */
struct partial_symtab **dependencies;
int number_of_dependencies;
/* If NULL, this is an ordinary partial symbol table. /* If NULL, this is an ordinary partial symbol table.
If non-NULL, this holds a single includer of this partial symbol If non-NULL, this holds a single includer of this partial symbol
@ -153,6 +140,19 @@ struct partial_symtab
struct partial_symtab *user; struct partial_symtab *user;
/* Array of pointers to all of the partial_symtab's which this one
depends on. Since this array can only be set to previous or
the current (?) psymtab, this dependency tree is guaranteed not
to have any loops. "depends on" means that symbols must be read
for the dependencies before being read for this psymtab; this is
for type references in stabs, where if foo.c includes foo.h, declarations
in foo.h may use type numbers defined in foo.c. For other debugging
formats there may be no need to use dependencies. */
struct partial_symtab **dependencies;
int number_of_dependencies;
/* Global symbol list. This list will be sorted after readin to /* Global symbol list. This list will be sorted after readin to
improve access. Binary search will be the usual method of improve access. Binary search will be the usual method of
finding a symbol within it. globals_offset is an integer offset finding a symbol within it. globals_offset is an integer offset