This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [RFA] Unify target header macros
- From: Joel Brobecker <brobecker at adacore dot com>
- To: Pierre Muller <muller at ics dot u-strasbg dot fr>
- Cc: gdb-patches at sourceware dot org
- Date: Mon, 11 May 2009 11:17:33 +0200
- Subject: Re: [RFA] Unify target header macros
- References: <005a01c9d18a$77d3c8b0$677b5a10$@u-strasbg.fr>
> In particular, I suspect that some of the comments
> that are changed by that patch are obsolete...
It would be nice to revisit these comments in the future, but in the
meantime, it's sufficient to keep their content in sync with the code.
So this is OK for now.
> 2009-05-10 Pierre Muller <muller.u-strasbg.fr>
>
> Unify target macros.
>
> * target.h (STOPPED_BY_WATCHPOINT): Delete, replaced by ...
> (target_stoppped_by_watchpoint): New macro.
> (HAVE_STEPPABLE_WATCHPOINT): Delete, replaced by ...
> (target_have_steppable_watchpoint): New macro.
> (HAVE_CONTINUABLE_WATCHPOINT): Delete, replace by ...
> (target_have_continuable_watchpoint): New macro.
> (TARGET_CAN_USE_HARDWARE_WATCHPOINT):Delete, replaced by ...
> (target_can_use_hardware_watchpoint): New macro.
> (TARGET_REGION_OK_FOR_HW_WATCHPOINT):Delete, replaced by ...
> (target_region_ok_for_hw_watchpoint): New macro.
>
> * breakpoint.c (update_watchpoint): Use new macros.
> (bpstat_alloc): Likewise.
> (create_breakpoint): Likewise.
> (watch_command_1): Likewise.
> (can_use_hardware_watchpoint): Likewise.
> (do_enable_breakpoint): Likewise.
> * infrun.c (handle_inferior_event): Adapt to new macros.
> * mips-tdep.c (mips_gdbarch_init): Update comments.
> * procfs.c (procfs_set_watchpoint): Update comment.
> (procfs_insert_watchpoint): Adapt to new macros.
> * remote-m32r-sdi.c (m32r_stop):
> * remote-mips.c (mips_remove_breakpoint):
> * target.c (debug_to_region_ok_for_hw_watchpoint): Update to new
> macros.
> (debug_to_stopped_by_watchpoint): Likewise.
Looks good to me.
Thanks for doing this.
--
Joel