Rename to allow_hw_watchpoint_multi_tests
authorTom Tromey <tom@tromey.com>
Sun, 8 Jan 2023 17:57:35 +0000 (10:57 -0700)
committerTom Tromey <tom@tromey.com>
Fri, 13 Jan 2023 20:18:57 +0000 (13:18 -0700)
This changes skip_hw_watchpoint_multi_tests to invert the sense, and
renames it to allow_hw_watchpoint_multi_tests.

gdb/testsuite/gdb.base/break-idempotent.exp
gdb/testsuite/gdb.base/watchpoint.exp
gdb/testsuite/gdb.multi/watchpoint-multi.exp
gdb/testsuite/gdb.threads/local-watch-wrong-thread.exp
gdb/testsuite/gdb.threads/watchthreads-reorder.exp
gdb/testsuite/gdb.threads/watchthreads.exp
gdb/testsuite/lib/gdb.exp

index e967b563ddaa6bd99bda6e14a75cfe4947312413..4d74bb54a1e7f1fa1399650898d0538e73de7276 100644 (file)
@@ -185,7 +185,7 @@ foreach_with_prefix pie { "nopie" "pie" } {
        }
 
        if {[allow_hw_watchpoint_access_tests]
-           && ![skip_hw_watchpoint_multi_tests]} {
+           && [allow_hw_watchpoint_multi_tests]} {
            test_break $always_inserted "rwatch"
            test_break $always_inserted "awatch"
        }
index c47e083d083075fca3ca1a07064fe918a583b052..5f5e7923c7989200af54638376e3c750a7815c99 100644 (file)
@@ -771,7 +771,7 @@ proc test_inaccessible_watchpoint {} {
 
        # This step requires two HW watchpoints.  Since some platforms only
        # have a single one, accept either SW or HW watchpoint in this case.
-       if {[skip_hw_watchpoint_multi_tests]} {
+       if {![allow_hw_watchpoint_multi_tests]} {
            set watchpoint_msg "(Watchpoint|Hardware watchpoint)"
        }
 
index 69996ba88d25ae68587c357c304e630584b6de6d..109defa864a32ad6b871bde9a9f97e425fa9cd7c 100644 (file)
@@ -62,7 +62,7 @@ gdb_breakpoint "marker_exit"
 
 gdb_test "inferior 1" "witching to inferior 1 .*" "switch back to inferior 1"
 
-if [skip_hw_watchpoint_multi_tests] {
+if {![allow_hw_watchpoint_multi_tests]} {
     # On single hardware watchpoint at least test the watchpoint in inferior
     # 2 is not hit.
 } else {
index 2f627a0598053cc42a7089fd6fc1743ce997ce10..d58c777cef6b17d5a6358f69a3ae2db4fb058979 100644 (file)
@@ -19,7 +19,7 @@
 # thread other than the thread the local watchpoint was set in stops
 # for a breakpoint.
 
-require !skip_hw_watchpoint_multi_tests
+require allow_hw_watchpoint_multi_tests
 
 standard_testfile
 if {[gdb_compile_pthreads \
index d5de2c68684b99cb57436efe6647f94a894e67c3..68409eff325d1345a4fbf3782b067ccf8c11f1da 100644 (file)
@@ -25,7 +25,7 @@
 # could be assigned during continuation of a thread with pending SIGTRAP to the
 # different/new watchpoint, just based on the watchpoint/debug register number.
 
-require allow_hw_watchpoint_access_tests !skip_hw_watchpoint_multi_tests
+require allow_hw_watchpoint_access_tests allow_hw_watchpoint_multi_tests
 if {![istarget *-*-linux*]} {
     return 0
 }
index d2a95bf7d5cb5f7a7e0a16d73ab7679f3401e466..1aa7957c867fe4cbd4618fa0aeea494d9ddb16e7 100644 (file)
@@ -20,7 +20,7 @@
 
 # This test verifies that a watchpoint is detected in the proper thread
 # so the test is only meaningful on a system with hardware watchpoints.
-require !skip_hw_watchpoint_multi_tests
+require allow_hw_watchpoint_multi_tests
 
 proc target_no_stopped_data { } {
     return [istarget s390*-*-*]
index 13d1ae881b57c0f583d7b085a5362a37dcd8a682..446f89a8a25a4a479d4b783a2eea376d4b2b13fb 100644 (file)
@@ -4064,21 +4064,21 @@ proc skip_hw_watchpoint_tests {} {
     return 1
 }
 
-# Return a 1 if we should skip tests that require *multiple* hardware
+# Return a 1 if we should run tests that require *multiple* hardware
 # watchpoints to be active at the same time
 
-proc skip_hw_watchpoint_multi_tests {} {
+proc allow_hw_watchpoint_multi_tests {} {
     if { [skip_hw_watchpoint_tests] } {
-       return 1
+       return 0
     }
 
     # These targets support just a single hardware watchpoint
     if { [istarget "arm*-*-*"]
         || [istarget "powerpc*-*-linux*"] } {
-       return 1
+       return 0
     }
 
-    return 0
+    return 1
 }
 
 # Return a 1 if we should run tests that require read/access watchpoints