public inbox for binutils@sourceware.org
 help / color / mirror / Atom feed
* [PATCH 10/15] MIPS/GAS/test: Remove FPU and SD insns from LD tests
@ 2010-10-03 19:41 Maciej W. Rozycki
  2010-10-10  9:53 ` Richard Sandiford
  0 siblings, 1 reply; 3+ messages in thread
From: Maciej W. Rozycki @ 2010-10-03 19:41 UTC (permalink / raw)
  To: binutils

Hi,

 In preparation for full coverage of FPU and SD instructions, these are 
removed from the LD tests.  This will reduce the number of dump 
variations.

2010-10-03  Maciej W. Rozycki  <macro@linux-mips.org>

	gas/testsuite/
	* gas/mips/ld.s: Remove "l.d", "s.d" and "sd" instructions.
	* gas/mips/ld.d: Adjust accordingly.
	* gas/mips/ld-ilocks.d: Likewise

 OK to apply?

  Maciej

binutils-2.20.51-20100925-mips-gas-test-ld-only.patch
Index: binutils-2.20.51/gas/testsuite/gas/mips/ld-ilocks.d
===================================================================
--- binutils-2.20.51.orig/gas/testsuite/gas/mips/ld-ilocks.d
+++ binutils-2.20.51/gas/testsuite/gas/mips/ld-ilocks.d
@@ -563,59 +563,4 @@ Disassembly of section \.text:
 [ 	]*[0-9a-f]+: (R_MIPS_LO16|REFLO)	\.sbss(\+0xfffff000)?
 [0-9a-f]+ <[^>]*> lw	a1,-(23127|19031)\(at\)
 [ 	]*[0-9a-f]+: (R_MIPS_LO16|REFLO)	\.sbss(\+0xfffff000)?
-[0-9a-f]+ <[^>]*> lwc1	\$f[45],0\(zero\)
-[0-9a-f]+ <[^>]*> lwc1	\$f[45],4\(zero\)
-[0-9a-f]+ <[^>]*> lwc1	\$f[45],1\(zero\)
-[0-9a-f]+ <[^>]*> lwc1	\$f[45],5\(zero\)
-[0-9a-f]+ <[^>]*> lui	at,0x1
-[0-9a-f]+ <[^>]*> lwc1	\$f[45],-32768\(at\)
-[0-9a-f]+ <[^>]*> lwc1	\$f[45],-32764\(at\)
-[0-9a-f]+ <[^>]*> lwc1	\$f[45],-32768\(zero\)
-[0-9a-f]+ <[^>]*> lwc1	\$f[45],-32764\(zero\)
-[0-9a-f]+ <[^>]*> lwc1	\$f[45],0\(a1\)
-[0-9a-f]+ <[^>]*> lwc1	\$f[45],4\(a1\)
-[0-9a-f]+ <[^>]*> lwc1	\$f[45],1\(a1\)
-[0-9a-f]+ <[^>]*> lwc1	\$f[45],5\(a1\)
-[0-9a-f]+ <[^>]*> lui	at,0x1
-[0-9a-f]+ <[^>]*> addu	at,a1,at
-[0-9a-f]+ <[^>]*> lwc1	\$f[45],-32768\(at\)
-[0-9a-f]+ <[^>]*> lwc1	\$f[45],-32764\(at\)
-[0-9a-f]+ <[^>]*> lwc1	\$f[45],-32768\(a1\)
-[0-9a-f]+ <[^>]*> lwc1	\$f[45],-32764\(a1\)
-[0-9a-f]+ <[^>]*> lui	at,0x2
-[ 	]*[0-9a-f]+: (R_MIPS_HI16|REFHI)	small_external_common
-[0-9a-f]+ <[^>]*> addu	at,a1,at
-[0-9a-f]+ <[^>]*> lwc1	\$f[45],-23131\(at\)
-[ 	]*[0-9a-f]+: (R_MIPS_LO16|REFLO)	small_external_common
-[0-9a-f]+ <[^>]*> lwc1	\$f[45],-23127\(at\)
-[ 	]*[0-9a-f]+: (R_MIPS_LO16|REFLO)	small_external_common
-[0-9a-f]+ <[^>]*> nop
-[0-9a-f]+ <[^>]*> swc1	\$f[45],0\(zero\)
-[0-9a-f]+ <[^>]*> swc1	\$f[45],4\(zero\)
-[0-9a-f]+ <[^>]*> swc1	\$f[45],1\(zero\)
-[0-9a-f]+ <[^>]*> swc1	\$f[45],5\(zero\)
-[0-9a-f]+ <[^>]*> lui	at,0x1
-[0-9a-f]+ <[^>]*> swc1	\$f[45],-32768\(at\)
-[0-9a-f]+ <[^>]*> swc1	\$f[45],-32764\(at\)
-[0-9a-f]+ <[^>]*> swc1	\$f[45],-32768\(zero\)
-[0-9a-f]+ <[^>]*> swc1	\$f[45],-32764\(zero\)
-[0-9a-f]+ <[^>]*> swc1	\$f[45],0\(a1\)
-[0-9a-f]+ <[^>]*> swc1	\$f[45],4\(a1\)
-[0-9a-f]+ <[^>]*> swc1	\$f[45],1\(a1\)
-[0-9a-f]+ <[^>]*> swc1	\$f[45],5\(a1\)
-[0-9a-f]+ <[^>]*> lui	at,0x1
-[0-9a-f]+ <[^>]*> addu	at,a1,at
-[0-9a-f]+ <[^>]*> swc1	\$f[45],-32768\(at\)
-[0-9a-f]+ <[^>]*> swc1	\$f[45],-32764\(at\)
-[0-9a-f]+ <[^>]*> swc1	\$f[45],-32768\(a1\)
-[0-9a-f]+ <[^>]*> swc1	\$f[45],-32764\(a1\)
-[0-9a-f]+ <[^>]*> lui	at,0x2
-[ 	]*[0-9a-f]+: (R_MIPS_HI16|REFHI)	big_external_common
-[0-9a-f]+ <[^>]*> addu	at,a1,at
-[0-9a-f]+ <[^>]*> swc1	\$f[45],-23131\(at\)
-[ 	]*[0-9a-f]+: (R_MIPS_LO16|REFLO)	big_external_common
-[0-9a-f]+ <[^>]*> swc1	\$f[45],-23127\(at\)
-[ 	]*[0-9a-f]+: (R_MIPS_LO16|REFLO)	big_external_common
-[0-9a-f]+ <[^>]*> sw	a0,0\(zero\)
-[0-9a-f]+ <[^>]*> sw	a1,4\(zero\)
 	\.\.\.
Index: binutils-2.20.51/gas/testsuite/gas/mips/ld.d
===================================================================
--- binutils-2.20.51.orig/gas/testsuite/gas/mips/ld.d
+++ binutils-2.20.51/gas/testsuite/gas/mips/ld.d
@@ -571,59 +571,4 @@ Disassembly of section \.text:
 [ 	]*[0-9a-f]+: (R_MIPS_LO16|REFLO)	\.sbss(\+0xfffff000)?
 [0-9a-f]+ <[^>]*> lw	a1,-(23127|19031)\(at\)
 [ 	]*[0-9a-f]+: (R_MIPS_LO16|REFLO)	\.sbss(\+0xfffff000)?
-[0-9a-f]+ <[^>]*> lwc1	\$f[45],0\(zero\)
-[0-9a-f]+ <[^>]*> lwc1	\$f[45],4\(zero\)
-[0-9a-f]+ <[^>]*> lwc1	\$f[45],1\(zero\)
-[0-9a-f]+ <[^>]*> lwc1	\$f[45],5\(zero\)
-[0-9a-f]+ <[^>]*> lui	at,0x1
-[0-9a-f]+ <[^>]*> lwc1	\$f[45],-32768\(at\)
-[0-9a-f]+ <[^>]*> lwc1	\$f[45],-32764\(at\)
-[0-9a-f]+ <[^>]*> lwc1	\$f[45],-32768\(zero\)
-[0-9a-f]+ <[^>]*> lwc1	\$f[45],-32764\(zero\)
-[0-9a-f]+ <[^>]*> lwc1	\$f[45],0\(a1\)
-[0-9a-f]+ <[^>]*> lwc1	\$f[45],4\(a1\)
-[0-9a-f]+ <[^>]*> lwc1	\$f[45],1\(a1\)
-[0-9a-f]+ <[^>]*> lwc1	\$f[45],5\(a1\)
-[0-9a-f]+ <[^>]*> lui	at,0x1
-[0-9a-f]+ <[^>]*> addu	at,a1,at
-[0-9a-f]+ <[^>]*> lwc1	\$f[45],-32768\(at\)
-[0-9a-f]+ <[^>]*> lwc1	\$f[45],-32764\(at\)
-[0-9a-f]+ <[^>]*> lwc1	\$f[45],-32768\(a1\)
-[0-9a-f]+ <[^>]*> lwc1	\$f[45],-32764\(a1\)
-[0-9a-f]+ <[^>]*> lui	at,0x2
-[ 	]*[0-9a-f]+: (R_MIPS_HI16|REFHI)	small_external_common
-[0-9a-f]+ <[^>]*> addu	at,a1,at
-[0-9a-f]+ <[^>]*> lwc1	\$f[45],-23131\(at\)
-[ 	]*[0-9a-f]+: (R_MIPS_LO16|REFLO)	small_external_common
-[0-9a-f]+ <[^>]*> lwc1	\$f[45],-23127\(at\)
-[ 	]*[0-9a-f]+: (R_MIPS_LO16|REFLO)	small_external_common
-[0-9a-f]+ <[^>]*> nop
-[0-9a-f]+ <[^>]*> swc1	\$f[45],0\(zero\)
-[0-9a-f]+ <[^>]*> swc1	\$f[45],4\(zero\)
-[0-9a-f]+ <[^>]*> swc1	\$f[45],1\(zero\)
-[0-9a-f]+ <[^>]*> swc1	\$f[45],5\(zero\)
-[0-9a-f]+ <[^>]*> lui	at,0x1
-[0-9a-f]+ <[^>]*> swc1	\$f[45],-32768\(at\)
-[0-9a-f]+ <[^>]*> swc1	\$f[45],-32764\(at\)
-[0-9a-f]+ <[^>]*> swc1	\$f[45],-32768\(zero\)
-[0-9a-f]+ <[^>]*> swc1	\$f[45],-32764\(zero\)
-[0-9a-f]+ <[^>]*> swc1	\$f[45],0\(a1\)
-[0-9a-f]+ <[^>]*> swc1	\$f[45],4\(a1\)
-[0-9a-f]+ <[^>]*> swc1	\$f[45],1\(a1\)
-[0-9a-f]+ <[^>]*> swc1	\$f[45],5\(a1\)
-[0-9a-f]+ <[^>]*> lui	at,0x1
-[0-9a-f]+ <[^>]*> addu	at,a1,at
-[0-9a-f]+ <[^>]*> swc1	\$f[45],-32768\(at\)
-[0-9a-f]+ <[^>]*> swc1	\$f[45],-32764\(at\)
-[0-9a-f]+ <[^>]*> swc1	\$f[45],-32768\(a1\)
-[0-9a-f]+ <[^>]*> swc1	\$f[45],-32764\(a1\)
-[0-9a-f]+ <[^>]*> lui	at,0x2
-[ 	]*[0-9a-f]+: (R_MIPS_HI16|REFHI)	big_external_common
-[0-9a-f]+ <[^>]*> addu	at,a1,at
-[0-9a-f]+ <[^>]*> swc1	\$f[45],-23131\(at\)
-[ 	]*[0-9a-f]+: (R_MIPS_LO16|REFLO)	big_external_common
-[0-9a-f]+ <[^>]*> swc1	\$f[45],-23127\(at\)
-[ 	]*[0-9a-f]+: (R_MIPS_LO16|REFLO)	big_external_common
-[0-9a-f]+ <[^>]*> sw	a0,0\(zero\)
-[0-9a-f]+ <[^>]*> sw	a1,4\(zero\)
 	\.\.\.
Index: binutils-2.20.51/gas/testsuite/gas/mips/ld.s
===================================================================
--- binutils-2.20.51.orig/gas/testsuite/gas/mips/ld.s
+++ binutils-2.20.51/gas/testsuite/gas/mips/ld.s
@@ -113,32 +113,6 @@ data_label:
 	ld	$4,big_local_common+0x1a5a5($5)
 	ld	$4,small_local_common+0x1a5a5($5)
 
-# l.d and s.d are sort of like ld.
-	l.d	$f4,0
-	l.d	$f4,1
-	l.d	$f4,0x8000
-	l.d	$f4,-0x8000
-	l.d	$f4,0($5)
-	l.d	$f4,1($5)
-	l.d	$f4,0x8000($5)
-	l.d	$f4,-0x8000($5)
-	l.d	$f4,small_external_common+0x1a5a5($5)
-	# Little endian will insert a nop here.
-	# We put it in explicitly so that big and little endian are similar.
-	nop
-	s.d	$f4,0
-	s.d	$f4,1
-	s.d	$f4,0x8000
-	s.d	$f4,-0x8000
-	s.d	$f4,0($5)
-	s.d	$f4,1($5)
-	s.d	$f4,0x8000($5)
-	s.d	$f4,-0x8000($5)
-	s.d	$f4,big_external_common+0x1a5a5($5)
-
-# sd is handled like ld.  Sanity check it.
-	sd	$4,0
-
 # Force at least 8 (non-delay-slot) zero bytes, to make 'objdump' print ...
 	.align	2
 	.space	8

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH 10/15] MIPS/GAS/test: Remove FPU and SD insns from LD tests
  2010-10-03 19:41 [PATCH 10/15] MIPS/GAS/test: Remove FPU and SD insns from LD tests Maciej W. Rozycki
@ 2010-10-10  9:53 ` Richard Sandiford
  2010-10-24  7:35   ` Maciej W. Rozycki
  0 siblings, 1 reply; 3+ messages in thread
From: Richard Sandiford @ 2010-10-10  9:53 UTC (permalink / raw)
  To: Maciej W. Rozycki; +Cc: binutils

"Maciej W. Rozycki" <macro@linux-mips.org> writes:
> 2010-10-03  Maciej W. Rozycki  <macro@linux-mips.org>
>
> 	gas/testsuite/
> 	* gas/mips/ld.s: Remove "l.d", "s.d" and "sd" instructions.
> 	* gas/mips/ld.d: Adjust accordingly.
> 	* gas/mips/ld-ilocks.d: Likewise

OK in combination with patch 15, thanks.

Richard

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH 10/15] MIPS/GAS/test: Remove FPU and SD insns from LD tests
  2010-10-10  9:53 ` Richard Sandiford
@ 2010-10-24  7:35   ` Maciej W. Rozycki
  0 siblings, 0 replies; 3+ messages in thread
From: Maciej W. Rozycki @ 2010-10-24  7:35 UTC (permalink / raw)
  To: Richard Sandiford; +Cc: binutils

On Sun, 10 Oct 2010, Richard Sandiford wrote:

> > 	gas/testsuite/
> > 	* gas/mips/ld.s: Remove "l.d", "s.d" and "sd" instructions.
> > 	* gas/mips/ld.d: Adjust accordingly.
> > 	* gas/mips/ld-ilocks.d: Likewise
> 
> OK in combination with patch 15, thanks.

 That's the whole point of this change.  Applied now, thanks.

  Maciej

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2010-10-24  7:35 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2010-10-03 19:41 [PATCH 10/15] MIPS/GAS/test: Remove FPU and SD insns from LD tests Maciej W. Rozycki
2010-10-10  9:53 ` Richard Sandiford
2010-10-24  7:35   ` Maciej W. Rozycki

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).