[PATCH 3/4] Plumb enum remove_bp_reason all the way to target_remove_breakpoint
Yao Qi
qiyaoltc@gmail.com
Mon Feb 15 09:12:00 GMT 2016
Pedro Alves <palves@redhat.com> writes:
Hi Pedro,
> So the target knows whether we're detaching breakpoints.
> Nothing uses the parameter in this patch yet.
>
> gdb/ChangeLog:
> 2016-02-14 Pedro Alves <palves@redhat.com>
>
> * break-catch-sig.c (signal_catchpoint_remove_location): Adjust
> interface.
> * break-catch-syscall.c (remove_catch_syscall):
> * breakpoint.c (enum remove_bp_reason): Moved to breakpoint.h.
> (remove_breakpoint_1): Pass 'reason' down.
> (remove_catch_fork, remove_catch_vfork, remove_catch_solib)
> (remove_catch_exec, remove_watchpoint, remove_masked_watchpoint)
> (base_breakpoint_remove_location, bkpt_remove_location)
> (bkpt_probe_remove_location, bkpt_probe_remove_location): Adjust
> interface.
> * breakpoint.h (enum remove_bp_reason): Moved here from
> breakpoint.c.
> (struct breakpoint_ops) <remove_location>: Add 'reason' parameter.
> * corelow.c (core_remove_breakpoint): New function.
> (init_core_ops): Install it as to_remove_breakpoint method.
> * exec.c (exec_remove_breakpoint): New function.
> (init_exec_ops): Install it as to_remove_breakpoint method.
> * mem-break.c (memory_remove_breakpoint): Adjust interface.
> * record-btrace.c (record_btrace_remove_breakpoint): Adjust
> interface.
> * record-full.c (record_full_remove_breakpoint)
> (record_full_core_remove_breakpoint): Adjust interface.
> * remote.c (remote_remove_breakpoint): Adjust interface.
> * target-debug.h (target_debug_print_enum_remove_bp_reason): New
> macro.
> * target-delegates.c: Regenerate.
> * target.c (target_remove_breakpoint): Add 'reason' parameter.
> * target.h (struct target_ops) <to_remove_breakpoint>: Add
> 'reason' parameter.
> (target_remove_breakpoint, memory_remove_breakpoint): Add 'reason'
> parameter.
We also need to add 'reason' in m32r_remove_breakpoint,
mips_remove_breakpoint and procfs_remove_breakpoint, otherwise the patch
breaks the all-targets build.
This patch series look good to me.
--
Yao (齐尧)
More information about the Gdb-patches
mailing list