[PATCH] Improve mips sim testsuite

Thiemo Seufer ths@networkno.de
Wed Nov 8 12:02:00 GMT 2006


Hello All,

I applied the appended patch, it improves the mips sim testsuite a bit.


Thiemo


2006-11-08  Thiemo Seufer  <ths@mips.com>

	* basic.exp: Fix spelling in comment. Use canonical form of target
	patterns. Run DSP test only for DSP-capable ISAs. Check also mips32r2
	and mips64r2 if supported by the target.


Index: basic.exp
===================================================================
RCS file: /cvs/src/src/sim/testsuite/sim/mips/basic.exp,v
retrieving revision 1.6
diff -u -p -r1.6 basic.exp
--- basic.exp	14 Dec 2005 23:09:26 -0000	1.6
+++ basic.exp	8 Nov 2006 11:49:30 -0000
@@ -1,6 +1,6 @@
 # MIPS simulator instruction tests
 
-# As gross as it is, we unset the linker script specifid by the target
+# As gross as it is, we unset the linker script specified by the target
 # board.  The MIPS libgloss linker scripts include libgcc (and possibly
 # other libraries), which the linker (used to link these tests rather
 # than the compiler) can't necessarily find.
@@ -32,23 +32,27 @@ proc run_hilo_test {testfile models nops
 }
 
 
-# Only test mips*-elf (e.g., no mips-linux), and only test if the target
+# Only test mips*-*-elf (e.g., no mips*-*-linux), and only test if the target
 # board really is a simulator (sim tests don't work on real HW).
-if {[istarget mips*-elf] && [board_info target exists is_simulator]} {
+if {[istarget mips*-*-elf] && [board_info target exists is_simulator]} {
 
-    if {[istarget mipsisa64sb1*-elf]} {
+    set dspmodels ""
+
+    if {[istarget mipsisa64sb1*-*-elf]} {
 	set models "sb1"
 	set submodels "mips1 mips2 mips3 mips4 mips32 mips64"
-    } elseif {[istarget mipsisa64*-elf]} {
-	set models "mips32 mips64"
+    } elseif {[istarget mipsisa64*-*-elf]} {
+	set models "mips32 mips64 mips32r2 mips64r2"
 	set submodels "mips1 mips2 mips3 mips4"
-    } elseif {[istarget mipsisa32*-elf]} {
-	set models "mips32"
+	append dspmodels " mips32r2 mips64r2"
+    } elseif {[istarget mipsisa32*-*-elf]} {
+	set models "mips32 mips32r2"
 	set submodels "mips1 mips2"
-    } elseif {[istarget mips64vr-*-elf] || [istarget mips64vrel-*-elf]} {
+	append dspmodels " mips32r2"
+    } elseif {[istarget mips64vr*-*-elf]} {
 	set models "vr4100 vr4111 vr4120 vr5000 vr5400 vr5500"
 	set submodels "mips1 mips2 mips3 mips4"
-    } elseif {[istarget mips64*-elf]} {
+    } elseif {[istarget mips64*-*-elf]} {
 	set models "mips3"
 	set submodels "mips1 mips2"
     } else {
@@ -72,5 +76,5 @@ if {[istarget mips*-elf] && [board_info 
     run_sim_test mdmx-ob.s $submodels
     run_sim_test mdmx-ob-sb1.s $submodels
 
-    run_sim_test mips32-dsp.s $models
+    run_sim_test mips32-dsp.s $dspmodels
 }



More information about the Gdb-patches mailing list