diff options
| author | Tom Tromey <tromey@adacore.com> | 2020-05-27 11:48:18 -0600 |
|---|---|---|
| committer | Tom Tromey <tromey@adacore.com> | 2020-05-27 11:48:19 -0600 |
| commit | af0b2a3e85df9f49a3528e5b7578fcf9412f1acc (patch) | |
| tree | 3bc4c4569634256e074e7260e4dd13e97723e98b /gdb/ada-varobj.c | |
| parent | 7d00ffecd2b16608f0bcfbc0402fec2cc228a3e7 (diff) | |
| download | gdb-af0b2a3e85df9f49a3528e5b7578fcf9412f1acc.zip gdb-af0b2a3e85df9f49a3528e5b7578fcf9412f1acc.tar.gz gdb-af0b2a3e85df9f49a3528e5b7578fcf9412f1acc.tar.bz2 | |
Inline abbrev lookup
Profiling showed that calls to abbrev_table::lookup_abbrev were "too
visible". As these are just forwarding calls to the hash table, this
patch inlines the lookup. Also, htab_find_with_hash is used, avoiding
another call.
The run previous to this had times of (see the first patch in the
series for an explanation):
gdb 1.69
libxul 2.02
Ada 2.52
This patch improves the times to:
gdb 1.64
libxul 1.99
Ada 2.47
gdb/ChangeLog
2020-05-27 Tom Tromey <tromey@adacore.com>
* dwarf2/abbrev.h (struct abbrev_table) <lookup_abbrev>: Inline.
Use htab_find_with_hash.
<add_abbrev>: Remove "abbrev_number" parameter.
* dwarf2/abbrev.c (abbrev_table::add_abbrev): Remove
"abbrev_number" parameter. Use htab_find_slot_with_hash.
(hash_abbrev): Add comment.
(abbrev_table::lookup_abbrev): Move to header file.
(abbrev_table::read): Update.
Diffstat (limited to 'gdb/ada-varobj.c')
0 files changed, 0 insertions, 0 deletions