[PATCH v2 06/12] MIPS/testsuite: Verify MIPS DSP ASE branch stepping

"Maciej W. Rozycki" <[email protected]>
Newsgroups gmane.comp.gdb.patches
Message-ID <[email protected]>
From: Maciej W. Rozycki <[email protected]>

Verify that breakpoints are correctly placed via `mips32_next_pc' while
single-stepping through MIPS DSP ASE branches.  No 64-bit support here.

Approved-By: Maciej W. Rozycki <[email protected]>
---
New change in v2.
---
 gdb/testsuite/gdb.arch/mips-allow.exp.tcl  |   19 +++++++
 gdb/testsuite/gdb.arch/mips-dsp-branch.c   |   69 +++++++++++++++++++++++++++++
 gdb/testsuite/gdb.arch/mips-dsp-branch.exp |   30 ++++++++++++
 3 files changed, 118 insertions(+)

gdb-mips-dsp-next-pc-test.diff
Index: binutils-gdb/gdb/testsuite/gdb.arch/mips-allow.exp.tcl
===================================================================
--- binutils-gdb.orig/gdb/testsuite/gdb.arch/mips-allow.exp.tcl
+++ binutils-gdb/gdb/testsuite/gdb.arch/mips-allow.exp.tcl
@@ -84,3 +84,22 @@ proc allow_mips2_branch_tests {} {
 	{-Wa,-fatal-warnings -mno-mips16 -mno-micromips
 	    -minterlink-compressed}]
 }
+
+# Check for MIPS DSP ASE instruction support.  These instructions may
+# be absent, such as with pre-MIPSr2 compilations or non-DSP hardware.
+proc allow_mips_dsp_ase_tests {} {
+    return [allow_target_tests "allow_mips_dsp_ase_tests" \
+	"MIPS DSP instruction support" "Illegal instruction" \
+	    {
+		int main() {
+		    asm volatile (
+			".set	push\n\t"
+			".set	noreorder\n\t"
+			".set	nomacro\n\t"
+			"rddsp	$0, 0\n\t"
+			".set	pop\n");
+		    return 0;
+		}
+	    } \
+	{-Wa,-fatal-warnings -mno-mips16 -minterlink-compressed -mdsp}]
+}
Index: binutils-gdb/gdb/testsuite/gdb.arch/mips-dsp-branch.c
===================================================================
--- /dev/null
+++ binutils-gdb/gdb/testsuite/gdb.arch/mips-dsp-branch.c
@@ -0,0 +1,69 @@
+/* This testcase is part of GDB, the GNU debugger.
+
+   Copyright (C) 2026 Free Software Foundation, Inc.
+
+   This program is free software; you can redistribute it and/or modify
+   it under the terms of the GNU General Public License as published by
+   the Free Software Foundation; either version 3 of the License, or
+   (at your option) any later version.
+
+   This program is distributed in the hope that it will be usefu,
+   but WITHOUT ANY WARRANTY; without even the implied warranty of
+   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+   GNU General Public License for more details.
+
+   You should have received a copy of the GNU General Public License
+   along with this program.  If not, see <http://www.gnu.org/licenses/>.  */
+
+/* Test single-stepping through MIPS DSP branch instructions.  */
+
+int
+test_mips_dsp_branch (void)
+{
+  /* Make 'err' available to the debugger to track the number of single
+     steps executed.  Use `volatile' to prevent the variable from being
+     optimized away.  */
+  volatile int err = -1;
+
+  int thr_one = 31;
+  int thr_two = 32;
+  int thr_thr = 33;
+
+  asm volatile (
+	".macro	b_test op\n\t"
+	"\\op	0f\n\t"
+	" nop\n\t"
+	"nop\n"
+	"0:\n\t"
+	".endm\n\t"
+
+	".set	push\n\t"
+	".set	noreorder\n\t"
+	".globl	step_start\n\t"
+	".type	step_start, @function\n"
+	"step_start:\n\t"			/* Units: steps.  */
+	"nop\n\t"				/* NOP:       1s  */
+	"wrdsp	%[thr_one], 1\n\t"		/* WRDSP:     1s  */
+	"b_test	bposge32\n\t"			/* Not taken: 2s  */
+	"wrdsp	%[thr_two], 1\n\t"		/* WRDSP:     1s  */
+	"b_test	bposge32\n\t"			/* Taken:     1s  */
+	"wrdsp	%[thr_thr], 1\n\t"		/* WRDSP:     1s  */
+	"b_test	bposge32\n\t"			/* Taken:     1s  */
+	"nop\n\t"				/* NOP:       1s  */
+	".globl	step_stop\n\t"			/* Total:     9s  */
+	".type	step_stop, @function\n"
+	"step_stop:\n\t"
+	".set pop\n"
+	:
+	: [thr_one] "r" (thr_one), [thr_two] "r" (thr_two),
+	  [thr_thr] "r" (thr_thr)
+	: "$dsp_po");
+
+  return err;
+}
+
+int
+main (void)
+{
+  return test_mips_dsp_branch ();
+}
Index: binutils-gdb/gdb/testsuite/gdb.arch/mips-dsp-branch.exp
===================================================================
--- /dev/null
+++ binutils-gdb/gdb/testsuite/gdb.arch/mips-dsp-branch.exp
@@ -0,0 +1,30 @@
+# Copyright (C) 2026 Free Software Foundation, Inc.
+
+# This program is free software; you can redistribute it and/or modify
+# it under the terms of the GNU General Public License as published by
+# the Free Software Foundation; either version 3 of the License, or
+# (at your option) any later version.
+#
+# This program is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+# GNU General Public License for more details.
+#
+# You should have received a copy of the GNU General Public License
+# along with this program.  If not, see <http://www.gnu.org/licenses/>.
+
+# Test single-stepping through MIPS DSP branch instructions.
+
+require {istarget "mips*-*-*"}
+
+source $srcdir/$subdir/mips-allow.exp.tcl
+
+require allow_mips_dsp_ase_tests
+
+standard_testfile
+
+set steps 9
+foreach flag {-mno-mips16 -minterlink-compressed -mdsp} {
+    lappend compile_flags "additional_flags=$flag"
+}
+source $srcdir/$subdir/mips-stepi.exp.tcl
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.