diff options
author | Weimin Pan <weimin.pan@oracle.com> | 2018-05-29 22:01:34 +0000 |
---|---|---|
committer | Weimin Pan <weimin.pan@oracle.com> | 2018-05-29 22:37:07 +0000 |
commit | 64cc34d87089ff24b43e0a8760be132b7afa6f34 (patch) | |
tree | 731d4904334e28a8ae4c888fac4c387299a5c089 /gdb/glibc-tdep.c | |
parent | e7ec8713ec12a633fcdfc5524c9ee050389b6f9d (diff) | |
download | gdb-64cc34d87089ff24b43e0a8760be132b7afa6f34.zip gdb-64cc34d87089ff24b43e0a8760be132b7afa6f34.tar.gz gdb-64cc34d87089ff24b43e0a8760be132b7afa6f34.tar.bz2 |
Removing lookup_minimal_symbol_and_objfile
Function lookup_minimal_symbol_and_objfile iterates on all objfiles and
calls lookup_minimal_symbol for each of them, effectively searching in all
objfiles. lookup_bound_minimal_symbol calls lookup_minimal_symbol with NULL,
which also effectively searches all objfiles. AFAIK, they do exactly the same
thing, so we can get rid of one (and lookup_minimal_symbol_and_objfile happens
to be the most inefficient because it ends up n^2 on the number of objfiles).
Tested in both aarch64-linux-gnu and amd64-linux-gnu. No regressions.
Diffstat (limited to 'gdb/glibc-tdep.c')
-rw-r--r-- | gdb/glibc-tdep.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/gdb/glibc-tdep.c b/gdb/glibc-tdep.c index 481e4b8..485cd8b 100644 --- a/gdb/glibc-tdep.c +++ b/gdb/glibc-tdep.c @@ -54,7 +54,7 @@ glibc_skip_solib_resolver (struct gdbarch *gdbarch, CORE_ADDR pc) debugging programs that use shared libraries. */ struct bound_minimal_symbol resolver - = lookup_minimal_symbol_and_objfile ("_dl_runtime_resolve"); + = lookup_bound_minimal_symbol ("_dl_runtime_resolve"); if (resolver.minsym) { |