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/printcmd.c | |
| parent | e7ec8713ec12a633fcdfc5524c9ee050389b6f9d (diff) | |
| download | binutils-gdb-64cc34d87089ff24b43e0a8760be132b7afa6f34.tar.gz | |
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/printcmd.c')
| -rw-r--r-- | gdb/printcmd.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/gdb/printcmd.c b/gdb/printcmd.c index 0dfea347533..d0037aec001 100644 --- a/gdb/printcmd.c +++ b/gdb/printcmd.c @@ -1549,7 +1549,7 @@ info_address_command (const char *exp, int from_tty) { struct bound_minimal_symbol msym; - msym = lookup_minimal_symbol_and_objfile (SYMBOL_LINKAGE_NAME (sym)); + msym = lookup_bound_minimal_symbol (SYMBOL_LINKAGE_NAME (sym)); if (msym.minsym == NULL) printf_filtered ("unresolved"); else |