espressif/binutils-gdb
1
0
Fork 0
mirror of https://github.com/espressif/binutils-gdb.git synced 2025-06-23 03:29:47 +08:00
Code Issues Packages Projects Releases Wiki Activity

Remove unused parameter in maybe_software_singlestep

Browse Source 
While working around, I noticed that the last parameter of
maybe_software_singlestep is never used.  This path removes
it.

Built on x86_64-linux-gnu and riscv64-linux-gnu.

gdb/ChangeLog:

	* infrun.c (maybe_software_singlestep): Remove unused PC
	parameter.
	(resume_1): Update calls to maybe_software_singlestep.
This commit is contained in:
Lancelot SIX
2021-07-09 00:09:11 +01:00
parent 661b504df9
commit 22b11ba924
1 changed files with 7 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

15
gdb/infrun.c
View File

@ -89,7 +89,7 @@ static void insert_step_resume_breakpoint_at_caller (struct frame_info *);
static void insert_longjmp_resume_breakpoint (struct gdbarch *, CORE_ADDR); static void insert_longjmp_resume_breakpoint (struct gdbarch *, CORE_ADDR);
static bool maybe_software_singlestep (struct gdbarch *gdbarch, CORE_ADDR pc); static bool maybe_software_singlestep (struct gdbarch *gdbarch);
static void resume (gdb_signal sig); static void resume (gdb_signal sig);
@ -2048,14 +2048,13 @@ set_schedlock_func (const char *args, int from_tty, struct cmd_list_element *c)
process. */ process. */
bool sched_multi = false; bool sched_multi = false;
/* Try to setup for software single stepping over the specified location. /* Try to setup for software single stepping. Return true if target_resume()
Return true if target_resume() should use hardware single step. should use hardware single step.
GDBARCH the current gdbarch. GDBARCH the current gdbarch. */
PC the location to step over. */
static bool static bool
maybe_software_singlestep (struct gdbarch *gdbarch, CORE_ADDR pc) maybe_software_singlestep (struct gdbarch *gdbarch)
{ {
bool hw_step = true; bool hw_step = true;
@ -2390,7 +2389,7 @@ resume_1 (enum gdb_signal sig)
set_step_over_info (regcache->aspace (), set_step_over_info (regcache->aspace (),
regcache_read_pc (regcache), 0, tp->global_num); regcache_read_pc (regcache), 0, tp->global_num);
step = maybe_software_singlestep (gdbarch, pc); step = maybe_software_singlestep (gdbarch);
insert_breakpoints (); insert_breakpoints ();
} }
@ -2409,7 +2408,7 @@ resume_1 (enum gdb_signal sig)
/* Do we need to do it the hard way, w/temp breakpoints? */ /* Do we need to do it the hard way, w/temp breakpoints? */
else if (step) else if (step)
step = maybe_software_singlestep (gdbarch, pc); step = maybe_software_singlestep (gdbarch);
/* Currently, our software single-step implementation leads to different /* Currently, our software single-step implementation leads to different
results than hardware single-stepping in one situation: when stepping results than hardware single-stepping in one situation: when stepping