public inbox for binutils@sourceware.org
 help / color / mirror / Atom feed
* [PATCH] MIPS/GAS: Correct branch relaxation message capitalisation
@ 2011-02-24 23:57 Maciej W. Rozycki
  2011-02-26  9:13 ` Richard Sandiford
  0 siblings, 1 reply; 3+ messages in thread
From: Maciej W. Rozycki @ 2011-02-24 23:57 UTC (permalink / raw)
  To: binutils; +Cc: Richard Sandiford

Hi,

 This change adjusts capitalisation of a message produced by branch 
relaxation.  It's needed for unified testing support of standard MIPS and 
microMIPS code and good anyway.  Regression-tested successfully with the 
mips-sde-elf and mips-linux-gnu targets.  OK to apply?

2011-02-24  Maciej W. Rozycki  <macro@codesourcery.com>

	gas/
	* config/tc-mips.c (md_convert_frag): Correct message
	capitalization.

	gas/testsuite/
	* gas/mips/relax-swap1.l: Adjust for message capitalization
	correction.
	* gas/mips/relax-swap2.l: Likewise.
	* gas/mips/relax.l: Likewise.

  Maciej

binutils-gas-mips-relax-msgformat.diff
Index: binutils-fsf-trunk-quilt/gas/config/tc-mips.c
===================================================================
--- binutils-fsf-trunk-quilt.orig/gas/config/tc-mips.c	2011-02-24 23:46:07.000000000 +0000
+++ binutils-fsf-trunk-quilt/gas/config/tc-mips.c	2011-02-24 23:46:08.000000000 +0000
@@ -14413,7 +14413,7 @@ md_convert_frag (bfd *abfd ATTRIBUTE_UNU
 	  int i;
 
 	  as_warn_where (fragp->fr_file, fragp->fr_line,
-			 _("relaxed out-of-range branch into a jump"));
+			 _("Relaxed out-of-range branch into a jump"));
 
 	  if (RELAX_BRANCH_UNCOND (fragp->fr_subtype))
 	    goto uncond;
Index: binutils-fsf-trunk-quilt/gas/testsuite/gas/mips/relax-swap1.l
===================================================================
--- binutils-fsf-trunk-quilt.orig/gas/testsuite/gas/mips/relax-swap1.l	2011-02-24 23:46:02.000000000 +0000
+++ binutils-fsf-trunk-quilt/gas/testsuite/gas/mips/relax-swap1.l	2011-02-24 23:46:08.000000000 +0000
@@ -1,24 +1,24 @@
 .*: Assembler messages:
-.*:9: Warning: relaxed out-of-range branch into a jump
-.*:14: Warning: relaxed out-of-range branch into a jump
-.*:19: Warning: relaxed out-of-range branch into a jump
-.*:24: Warning: relaxed out-of-range branch into a jump
-.*:28: Warning: relaxed out-of-range branch into a jump
-.*:33: Warning: relaxed out-of-range branch into a jump
-.*:37: Warning: relaxed out-of-range branch into a jump
-.*:42: Warning: relaxed out-of-range branch into a jump
-.*:46: Warning: relaxed out-of-range branch into a jump
-.*:51: Warning: relaxed out-of-range branch into a jump
-.*:55: Warning: relaxed out-of-range branch into a jump
-.*:62: Warning: relaxed out-of-range branch into a jump
-.*:68: Warning: relaxed out-of-range branch into a jump
-.*:73: Warning: relaxed out-of-range branch into a jump
-.*:79: Warning: relaxed out-of-range branch into a jump
-.*:85: Warning: relaxed out-of-range branch into a jump
-.*:96: Warning: relaxed out-of-range branch into a jump
-.*:101: Warning: relaxed out-of-range branch into a jump
-.*:106: Warning: relaxed out-of-range branch into a jump
-.*:111: Warning: relaxed out-of-range branch into a jump
-.*:116: Warning: relaxed out-of-range branch into a jump
-.*:121: Warning: relaxed out-of-range branch into a jump
-.*:126: Warning: relaxed out-of-range branch into a jump
+.*:9: Warning: Relaxed out-of-range branch into a jump
+.*:14: Warning: Relaxed out-of-range branch into a jump
+.*:19: Warning: Relaxed out-of-range branch into a jump
+.*:24: Warning: Relaxed out-of-range branch into a jump
+.*:28: Warning: Relaxed out-of-range branch into a jump
+.*:33: Warning: Relaxed out-of-range branch into a jump
+.*:37: Warning: Relaxed out-of-range branch into a jump
+.*:42: Warning: Relaxed out-of-range branch into a jump
+.*:46: Warning: Relaxed out-of-range branch into a jump
+.*:51: Warning: Relaxed out-of-range branch into a jump
+.*:55: Warning: Relaxed out-of-range branch into a jump
+.*:62: Warning: Relaxed out-of-range branch into a jump
+.*:68: Warning: Relaxed out-of-range branch into a jump
+.*:73: Warning: Relaxed out-of-range branch into a jump
+.*:79: Warning: Relaxed out-of-range branch into a jump
+.*:85: Warning: Relaxed out-of-range branch into a jump
+.*:96: Warning: Relaxed out-of-range branch into a jump
+.*:101: Warning: Relaxed out-of-range branch into a jump
+.*:106: Warning: Relaxed out-of-range branch into a jump
+.*:111: Warning: Relaxed out-of-range branch into a jump
+.*:116: Warning: Relaxed out-of-range branch into a jump
+.*:121: Warning: Relaxed out-of-range branch into a jump
+.*:126: Warning: Relaxed out-of-range branch into a jump
Index: binutils-fsf-trunk-quilt/gas/testsuite/gas/mips/relax-swap2.l
===================================================================
--- binutils-fsf-trunk-quilt.orig/gas/testsuite/gas/mips/relax-swap2.l	2011-02-24 23:46:02.000000000 +0000
+++ binutils-fsf-trunk-quilt/gas/testsuite/gas/mips/relax-swap2.l	2011-02-24 23:46:08.000000000 +0000
@@ -1,10 +1,10 @@
 .*: Assembler messages:
-.*:9: Warning: relaxed out-of-range branch into a jump
-.*:13: Warning: relaxed out-of-range branch into a jump
-.*:18: Warning: relaxed out-of-range branch into a jump
-.*:22: Warning: relaxed out-of-range branch into a jump
-.*:27: Warning: relaxed out-of-range branch into a jump
-.*:31: Warning: relaxed out-of-range branch into a jump
-.*:36: Warning: relaxed out-of-range branch into a jump
-.*:40: Warning: relaxed out-of-range branch into a jump
-.*:45: Warning: relaxed out-of-range branch into a jump
+.*:9: Warning: Relaxed out-of-range branch into a jump
+.*:13: Warning: Relaxed out-of-range branch into a jump
+.*:18: Warning: Relaxed out-of-range branch into a jump
+.*:22: Warning: Relaxed out-of-range branch into a jump
+.*:27: Warning: Relaxed out-of-range branch into a jump
+.*:31: Warning: Relaxed out-of-range branch into a jump
+.*:36: Warning: Relaxed out-of-range branch into a jump
+.*:40: Warning: Relaxed out-of-range branch into a jump
+.*:45: Warning: Relaxed out-of-range branch into a jump
Index: binutils-fsf-trunk-quilt/gas/testsuite/gas/mips/relax.l
===================================================================
--- binutils-fsf-trunk-quilt.orig/gas/testsuite/gas/mips/relax.l	2011-02-24 23:46:02.000000000 +0000
+++ binutils-fsf-trunk-quilt/gas/testsuite/gas/mips/relax.l	2011-02-24 23:46:08.000000000 +0000
@@ -1,45 +1,45 @@
 .*: Assembler messages:
-.*:5: Warning: relaxed out-of-range branch into a jump
-.*:6: Warning: relaxed out-of-range branch into a jump
-.*:7: Warning: relaxed out-of-range branch into a jump
-.*:8: Warning: relaxed out-of-range branch into a jump
-.*:9: Warning: relaxed out-of-range branch into a jump
-.*:10: Warning: relaxed out-of-range branch into a jump
-.*:11: Warning: relaxed out-of-range branch into a jump
-.*:12: Warning: relaxed out-of-range branch into a jump
-.*:13: Warning: relaxed out-of-range branch into a jump
-.*:14: Warning: relaxed out-of-range branch into a jump
-.*:16: Warning: relaxed out-of-range branch into a jump
-.*:17: Warning: relaxed out-of-range branch into a jump
-.*:19: Warning: relaxed out-of-range branch into a jump
-.*:20: Warning: relaxed out-of-range branch into a jump
-.*:21: Warning: relaxed out-of-range branch into a jump
-.*:22: Warning: relaxed out-of-range branch into a jump
-.*:23: Warning: relaxed out-of-range branch into a jump
-.*:24: Warning: relaxed out-of-range branch into a jump
-.*:25: Warning: relaxed out-of-range branch into a jump
-.*:26: Warning: relaxed out-of-range branch into a jump
-.*:28: Warning: relaxed out-of-range branch into a jump
-.*:29: Warning: relaxed out-of-range branch into a jump
-.*:33: Warning: relaxed out-of-range branch into a jump
-.*:34: Warning: relaxed out-of-range branch into a jump
-.*:35: Warning: relaxed out-of-range branch into a jump
-.*:36: Warning: relaxed out-of-range branch into a jump
-.*:37: Warning: relaxed out-of-range branch into a jump
-.*:38: Warning: relaxed out-of-range branch into a jump
-.*:39: Warning: relaxed out-of-range branch into a jump
-.*:40: Warning: relaxed out-of-range branch into a jump
-.*:41: Warning: relaxed out-of-range branch into a jump
-.*:42: Warning: relaxed out-of-range branch into a jump
-.*:44: Warning: relaxed out-of-range branch into a jump
-.*:45: Warning: relaxed out-of-range branch into a jump
-.*:47: Warning: relaxed out-of-range branch into a jump
-.*:48: Warning: relaxed out-of-range branch into a jump
-.*:49: Warning: relaxed out-of-range branch into a jump
-.*:50: Warning: relaxed out-of-range branch into a jump
-.*:51: Warning: relaxed out-of-range branch into a jump
-.*:52: Warning: relaxed out-of-range branch into a jump
-.*:53: Warning: relaxed out-of-range branch into a jump
-.*:54: Warning: relaxed out-of-range branch into a jump
-.*:56: Warning: relaxed out-of-range branch into a jump
-.*:57: Warning: relaxed out-of-range branch into a jump
+.*:5: Warning: Relaxed out-of-range branch into a jump
+.*:6: Warning: Relaxed out-of-range branch into a jump
+.*:7: Warning: Relaxed out-of-range branch into a jump
+.*:8: Warning: Relaxed out-of-range branch into a jump
+.*:9: Warning: Relaxed out-of-range branch into a jump
+.*:10: Warning: Relaxed out-of-range branch into a jump
+.*:11: Warning: Relaxed out-of-range branch into a jump
+.*:12: Warning: Relaxed out-of-range branch into a jump
+.*:13: Warning: Relaxed out-of-range branch into a jump
+.*:14: Warning: Relaxed out-of-range branch into a jump
+.*:16: Warning: Relaxed out-of-range branch into a jump
+.*:17: Warning: Relaxed out-of-range branch into a jump
+.*:19: Warning: Relaxed out-of-range branch into a jump
+.*:20: Warning: Relaxed out-of-range branch into a jump
+.*:21: Warning: Relaxed out-of-range branch into a jump
+.*:22: Warning: Relaxed out-of-range branch into a jump
+.*:23: Warning: Relaxed out-of-range branch into a jump
+.*:24: Warning: Relaxed out-of-range branch into a jump
+.*:25: Warning: Relaxed out-of-range branch into a jump
+.*:26: Warning: Relaxed out-of-range branch into a jump
+.*:28: Warning: Relaxed out-of-range branch into a jump
+.*:29: Warning: Relaxed out-of-range branch into a jump
+.*:33: Warning: Relaxed out-of-range branch into a jump
+.*:34: Warning: Relaxed out-of-range branch into a jump
+.*:35: Warning: Relaxed out-of-range branch into a jump
+.*:36: Warning: Relaxed out-of-range branch into a jump
+.*:37: Warning: Relaxed out-of-range branch into a jump
+.*:38: Warning: Relaxed out-of-range branch into a jump
+.*:39: Warning: Relaxed out-of-range branch into a jump
+.*:40: Warning: Relaxed out-of-range branch into a jump
+.*:41: Warning: Relaxed out-of-range branch into a jump
+.*:42: Warning: Relaxed out-of-range branch into a jump
+.*:44: Warning: Relaxed out-of-range branch into a jump
+.*:45: Warning: Relaxed out-of-range branch into a jump
+.*:47: Warning: Relaxed out-of-range branch into a jump
+.*:48: Warning: Relaxed out-of-range branch into a jump
+.*:49: Warning: Relaxed out-of-range branch into a jump
+.*:50: Warning: Relaxed out-of-range branch into a jump
+.*:51: Warning: Relaxed out-of-range branch into a jump
+.*:52: Warning: Relaxed out-of-range branch into a jump
+.*:53: Warning: Relaxed out-of-range branch into a jump
+.*:54: Warning: Relaxed out-of-range branch into a jump
+.*:56: Warning: Relaxed out-of-range branch into a jump
+.*:57: Warning: Relaxed out-of-range branch into a jump

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

* Re: [PATCH] MIPS/GAS: Correct branch relaxation message capitalisation
  2011-02-24 23:57 [PATCH] MIPS/GAS: Correct branch relaxation message capitalisation Maciej W. Rozycki
@ 2011-02-26  9:13 ` Richard Sandiford
  2011-02-28 15:34   ` Maciej W. Rozycki
  0 siblings, 1 reply; 3+ messages in thread
From: Richard Sandiford @ 2011-02-26  9:13 UTC (permalink / raw)
  To: Maciej W. Rozycki; +Cc: binutils

"Maciej W. Rozycki" <macro@codesourcery.com> writes:
> 2011-02-24  Maciej W. Rozycki  <macro@codesourcery.com>
>
> 	gas/
> 	* config/tc-mips.c (md_convert_frag): Correct message
> 	capitalization.
>
> 	gas/testsuite/
> 	* gas/mips/relax-swap1.l: Adjust for message capitalization
> 	correction.
> 	* gas/mips/relax-swap2.l: Likewise.
> 	* gas/mips/relax.l: Likewise.

OK, thanks.

Richard

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

* Re: [PATCH] MIPS/GAS: Correct branch relaxation message capitalisation
  2011-02-26  9:13 ` Richard Sandiford
@ 2011-02-28 15:34   ` Maciej W. Rozycki
  0 siblings, 0 replies; 3+ messages in thread
From: Maciej W. Rozycki @ 2011-02-28 15:34 UTC (permalink / raw)
  To: Richard Sandiford; +Cc: binutils

On Sat, 26 Feb 2011, Richard Sandiford wrote:

> > 2011-02-24  Maciej W. Rozycki  <macro@codesourcery.com>
> >
> > 	gas/
> > 	* config/tc-mips.c (md_convert_frag): Correct message
> > 	capitalization.
> >
> > 	gas/testsuite/
> > 	* gas/mips/relax-swap1.l: Adjust for message capitalization
> > 	correction.
> > 	* gas/mips/relax-swap2.l: Likewise.
> > 	* gas/mips/relax.l: Likewise.
> 
> OK, thanks.

 Done, thanks.

  Maciej

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

end of thread, other threads:[~2011-02-28 15:34 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-02-24 23:57 [PATCH] MIPS/GAS: Correct branch relaxation message capitalisation Maciej W. Rozycki
2011-02-26  9:13 ` Richard Sandiford
2011-02-28 15:34   ` 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).