Imagelibrary/binutils-gdb
1
0
Fork 1
mirror of https://github.com/bminor/binutils-gdb.git synced 2025-12-26 09:08:59 +00:00
Code Issues Packages Projects Releases Wiki Activity

Match demangled name in "skip"

Browse Source 
PR gdb/26598 notes that, before commit bcfe6157ca ("Use the linkage
name if it exists"), the "skip" command would match the demangled name
of a symbol, but now only matches the linkage name.

This patch fixes this regression.  I looked at all calls to
function_name_is_marked_for_skip, and only one used the linkage name.

2020-09-16  Tom Tromey  <tromey@adacore.com>

	PR gdb/26598:
	* infrun.c (fill_in_stop_func): Use find_pc_partial_function_sym.

gdb/testsuite/ChangeLog
2020-09-16  Tom Tromey  <tromey@adacore.com>

	PR gdb/26598:
	* gdb.base/skipcxx.exp: New file.
	* gdb.base/skipcxx.cc: New file.
This commit is contained in:
Tom Tromey
2020-09-16 09:49:36 -06:00
parent fe5ddfc3ee
commit fe83066292
5 changed files with 79 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
gdb/infrun.c
View File

@@ -4358,14 +4358,16 @@ fill_in_stop_func (struct gdbarch *gdbarch,
if (!ecs->stop_func_filled_in)
{
const block *block;
const general_symbol_info *gsi;
/* Don't care about return value; stop_func_start and stop_func_name
will both be 0 if it doesn't work. */
find_pc_partial_function (ecs->event_thread->suspend.stop_pc,
&ecs->stop_func_name,
&ecs->stop_func_start,
&ecs->stop_func_end,
&block);
find_pc_partial_function_sym (ecs->event_thread->suspend.stop_pc,
&gsi,
&ecs->stop_func_start,
&ecs->stop_func_end,
&block);
ecs->stop_func_name = gsi == nullptr ? nullptr : gsi->print_name ();
/* The call to find_pc_partial_function, above, will set
stop_func_start and stop_func_end to the start and end