[10/10] watchpoint.exp: Run tests twice: once with hw watchpoints enabled; another time with hw watchpoints disabled.

[Tom Tromey]

>>>>> "Pedro" == Pedro Alves <palves@redhat.com> writes:

Pedro> Finally, this makes tests that can run with or without hw
Pedro> watchpoints run twice, on targets that can do hw watchpoints.

Pedro> +# True if we're forcing no hardware watchpoints.
Pedro> +set no_hw 0
Pedro> +
Pedro>  # Prepare for watchpoint tests by setting up two breakpoints and one
Pedro>  # watchpoint.
Pedro>  #
Pedro> @@ -97,6 +100,10 @@ proc maybe_clean_restart { } {
Pedro>  	global testfile

Pedro>  	clean_restart $testfile
Pedro> +
Pedro> +	if {$no_hw} {

I think maybe_clean_restart needs "global no_hw".
I didn't see it elsewhere in the patches.

Pedro> @@ -108,6 +115,11 @@ proc maybe_reinitialize { } {
Pedro>  	global testfile

Pedro>  	clean_restart $testfile
Pedro> +
Pedro> +	if {$no_hw} {

Ditto.

Tom