[Bug breakpoints/11371] Request: add support for conditional watchpoint

["pedro at codesourcery dot com" <sourceware-bugzilla at sourceware dot org>]

------- Additional Comments From cvs-commit at gcc dot gnu dot org  2010-08-17 20:59 -------
Subject: Bug 11371

CVSROOT:	/cvs/src
Module name:	src
Changes by:	palves@sourceware.org	2010-08-17 20:59:04

Modified files:
	gdb            : ChangeLog breakpoint.c breakpoint.h 
	gdb/testsuite  : ChangeLog 
Added files:
	gdb/testsuite/gdb.base: watch-cond-infcall.c 
	                        watch-cond-infcall.exp 

Log message:
	gdb/
	2010-08-17  Pedro Alves  <pedro@codesourcery.com>
	
	PR breakpoints/11371
	
	* breakpoint.c (breakpoint_init_inferior): Decrement the
	location's reference count instead of deleting right away.
	(bpstat_free): Decrement the location's reference count.  Make
	static.
	(bpstat_copy): Increment the location's reference count.
	(bpstat_find_breakpoint): Adjust.
	(bpstat_num): Adjust.
	(print_it_typical): Adjust.  Use the breakpoint pointer in the
	bpstat instead of the location's owner.
	(bpstat_alloc): Remove const qualifier from the 'bl' parameter.
	Adjust to record the location's owner in the bpstat.
	(watchpoint_check): Use the breakpoint pointer in the bpstat
	instead of the location's owner.
	(bpstat_check_breakpoint_conditions): Don't handle
	bp_watchpoint_scope here.  Use the breakpoint pointer in the
	bpstat instead of the location's owner.
	(bpstat_stop_status): Defer inferior function calls to after
	building the bpstat list.  Handle bp_watchpoint_scope here.  Use
	the breakpoint pointer in the bpstat instead of the location's
	owner.
	(bpstat_what): Use the breakpoint pointer in the bpstat instead of
	the location's owner.
	(free_bp_location): Don't walk bpstats clearing locations.
	(incref_bp_location): New.
	(decref_bp_location): New.
	(breakpoint_auto_delete): Use the breakpoint pointer in the bpstat
	instead of the location's owner.
	(update_global_location_list): Clear the location's owner, and
	decrement the location's reference count instead of deleting it
	right away.
	(breakpoint_retire_moribund): Decrement the location's reference
	count instead of deleting it right away.
	(bpstat_remove_bp_location): Delete.
	(bpstat_remove_breakpoint): New.
	(bpstat_remove_bp_location_callback): Delete.
	(bpstat_remove_breakpoint_callback): New.
	(delete_breakpoint): Iterate over all threads' stop_bpstat's
	clearing references to the breakpoint that is being deleted.
	
	* breakpoint.h (struct bp_location) <refc>: New field.
	<owner>: Update comments.
	(bpstat_free): Delete declaration.
	(struct bpstats): Change the type of the breakpoint_at field to
	struct breakpoint point, from struct bp_location pointer.  Add new
	field bp_location_at.
	
	gdb/testsuite/
	2010-08-17  Jan Kratochvil  <jan.kratochvil@redhat.com>
	Pedro Alves  <pedro@codesourcery.com>
	
	PR breakpoints/11371
	
	* gdb.base/watch-cond-infcall.exp: New file.
	* gdb.base/watch-cond-infcall.c: New file.

Patches:
http://sourceware.org/cgi-bin/cvsweb.cgi/src/gdb/ChangeLog.diff?cvsroot=src&r1=1.12088&r2=1.12089
http://sourceware.org/cgi-bin/cvsweb.cgi/src/gdb/breakpoint.c.diff?cvsroot=src&r1=1.508&r2=1.509
http://sourceware.org/cgi-bin/cvsweb.cgi/src/gdb/breakpoint.h.diff?cvsroot=src&r1=1.124&r2=1.125
http://sourceware.org/cgi-bin/cvsweb.cgi/src/gdb/testsuite/ChangeLog.diff?cvsroot=src&r1=1.2411&r2=1.2412
http://sourceware.org/cgi-bin/cvsweb.cgi/src/gdb/testsuite/gdb.base/watch-cond-infcall.c.diff?cvsroot=src&r1=NONE&r2=1.1
http://sourceware.org/cgi-bin/cvsweb.cgi/src/gdb/testsuite/gdb.base/watch-cond-infcall.exp.diff?cvsroot=src&r1=NONE&r2=1.1



-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|cmoller at redhat dot com   |pedro at codesourcery dot
                   |                            |com


http://sourceware.org/bugzilla/show_bug.cgi?id=11371

------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.