From: Carl Love Date: Wed, 12 Oct 2022 20:58:25 +0000 (-0400) Subject: PowerPC, fix test gdb.base/watchpoint-stops-at-right-insn.exp X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=e610d38f83248ba2cbea4d8735c3f7ee5953a38b;p=binutils-gdb.git PowerPC, fix test gdb.base/watchpoint-stops-at-right-insn.exp Test gdb.base/watchpoint-stops-at-right-insn.exp generates 4 test errors on Power 9. The test uses the test [target_info exists gdb, no_hardware_watchpoints] to determine if the processor supports hardware watchpoints. The check only examines the processor type to determine if it supports hardware watchpoints. Note, the test works fine on Power 10. The PowerPC processors support hardware watchpoints with the exception of Power 9. The hardware watchpoint support is disabled on Power 9. The test skip_hw_watchpoint_tests must be used to correctly determine if the PowerPC processor supports hardware watchpoints. This patch replaces the [target_info exists gdb,no_hardware_watchpoints] with the skip_hw_watchpoint_tests_p check. With the patch, the test is disabled on Power 9 but runs on all other PowerPC processors. The patch has been tested on Power 9, Power 10 and X86-64 with no regression failures. --- diff --git a/gdb/testsuite/gdb.base/watchpoint-stops-at-right-insn.exp b/gdb/testsuite/gdb.base/watchpoint-stops-at-right-insn.exp index f9adb1f4f91..6c445d4cbd8 100644 --- a/gdb/testsuite/gdb.base/watchpoint-stops-at-right-insn.exp +++ b/gdb/testsuite/gdb.base/watchpoint-stops-at-right-insn.exp @@ -71,7 +71,7 @@ standard_testfile # No use testing this if we can't use hardware watchpoints. -if {[target_info exists gdb,no_hardware_watchpoints]} { +if {[skip_hw_watchpoint_tests]} { return -1 }