diff options
author | Simon Marchi <simon.marchi@ericsson.com> | 2016-02-11 14:08:53 -0500 |
---|---|---|
committer | Simon Marchi <simon.marchi@ericsson.com> | 2016-02-11 14:08:53 -0500 |
commit | 2ba163c8d19971010e22176cf688b3de80bd8f08 (patch) | |
tree | 70fbe6390ee3c8d3bf22d9ca0fd7f332a66eda17 | |
parent | 615234c107e5dcdefff78596971266e9346b3f76 (diff) | |
download | gdb-2ba163c8d19971010e22176cf688b3de80bd8f08.zip gdb-2ba163c8d19971010e22176cf688b3de80bd8f08.tar.gz gdb-2ba163c8d19971010e22176cf688b3de80bd8f08.tar.bz2 |
arm-tdep.c: Remove unused arm_displaced_step_copy_insn
This function is never used, since it is superseded by
arm_linux_displaced_step_copy_insn.
gdb/ChangeLog:
* arm-tdep.c (arm_displaced_step_copy_insn): Remove.
(ARM displaced stepping support): Remove reference to
arm_displaced_step_copy_insn in comment.
* arm-tdep.h (arm_displaced_step_copy_insn): Remove.
* arm-linux-tdep.c (arm_linux_displaced_step_copy_insn): Remove
reference to arm_displaced_step_copy_insn in comment.
-rw-r--r-- | gdb/ChangeLog | 9 | ||||
-rw-r--r-- | gdb/arm-linux-tdep.c | 3 | ||||
-rw-r--r-- | gdb/arm-tdep.c | 29 | ||||
-rw-r--r-- | gdb/arm-tdep.h | 3 |
4 files changed, 16 insertions, 28 deletions
diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 9a73992..f2707dc 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,5 +1,14 @@ 2016-02-11 Simon Marchi <simon.marchi@ericsson.com> + * arm-tdep.c (arm_displaced_step_copy_insn): Remove. + (ARM displaced stepping support): Remove reference to + arm_displaced_step_copy_insn in comment. + * arm-tdep.h (arm_displaced_step_copy_insn): Remove. + * arm-linux-tdep.c (arm_linux_displaced_step_copy_insn): Remove + reference to arm_displaced_step_copy_insn in comment. + +2016-02-11 Simon Marchi <simon.marchi@ericsson.com> + * arm-tdep.c (thumb_copy_unmodified_16bit): Change type of insn. (thumb_copy_b): Likewise. (arm_decode_b_bl_ldmstm): Likewise. diff --git a/gdb/arm-linux-tdep.c b/gdb/arm-linux-tdep.c index 3421f3b..5316963 100644 --- a/gdb/arm-linux-tdep.c +++ b/gdb/arm-linux-tdep.c @@ -1128,8 +1128,7 @@ arm_catch_kernel_helper_return (struct gdbarch *gdbarch, CORE_ADDR from, /* Linux-specific displaced step instruction copying function. Detects when the program has stepped into a Linux kernel helper routine (which must be - handled as a special case), falling back to arm_displaced_step_copy_insn() - if it hasn't. */ + handled as a special case). */ static struct displaced_step_closure * arm_linux_displaced_step_copy_insn (struct gdbarch *gdbarch, diff --git a/gdb/arm-tdep.c b/gdb/arm-tdep.c index 6ddc4bd..3e44db4 100644 --- a/gdb/arm-tdep.c +++ b/gdb/arm-tdep.c @@ -4307,13 +4307,12 @@ arm_adjust_breakpoint_address (struct gdbarch *gdbarch, CORE_ADDR bpaddr) Generally ARM displaced stepping works as follows: 1. When an instruction is to be single-stepped, it is first decoded by - arm_process_displaced_insn (called from arm_displaced_step_copy_insn). - Depending on the type of instruction, it is then copied to a scratch - location, possibly in a modified form. The copy_* set of functions - performs such modification, as necessary. A breakpoint is placed after - the modified instruction in the scratch space to return control to GDB. - Note in particular that instructions which modify the PC will no longer - do so after modification. + arm_process_displaced_insn. Depending on the type of instruction, it is + then copied to a scratch location, possibly in a modified form. The + copy_* set of functions performs such modification, as necessary. A + breakpoint is placed after the modified instruction in the scratch space + to return control to GDB. Note in particular that instructions which + modify the PC will no longer do so after modification. 2. The instruction is single-stepped, by setting the PC to the scratch location address, and resuming. Control returns to GDB when the @@ -7601,22 +7600,6 @@ arm_displaced_init_closure (struct gdbarch *gdbarch, CORE_ADDR from, paddress (gdbarch, from), paddress (gdbarch, to)); } -/* Entry point for copying an instruction into scratch space for displaced - stepping. */ - -struct displaced_step_closure * -arm_displaced_step_copy_insn (struct gdbarch *gdbarch, - CORE_ADDR from, CORE_ADDR to, - struct regcache *regs) -{ - struct displaced_step_closure *dsc = XNEW (struct displaced_step_closure); - - arm_process_displaced_insn (gdbarch, from, to, regs, dsc); - arm_displaced_init_closure (gdbarch, from, to, dsc); - - return dsc; -} - /* Entry point for cleaning things up after a displaced instruction has been single-stepped. */ diff --git a/gdb/arm-tdep.h b/gdb/arm-tdep.h index 1306cbb..e5d13bb 100644 --- a/gdb/arm-tdep.h +++ b/gdb/arm-tdep.h @@ -265,9 +265,6 @@ int arm_software_single_step (struct frame_info *); int arm_is_thumb (struct regcache *regcache); int arm_frame_is_thumb (struct frame_info *frame); -extern struct displaced_step_closure * - arm_displaced_step_copy_insn (struct gdbarch *, CORE_ADDR, CORE_ADDR, - struct regcache *); extern void arm_displaced_step_fixup (struct gdbarch *, struct displaced_step_closure *, CORE_ADDR, CORE_ADDR, struct regcache *); |