public inbox for binutils@sourceware.org
 help / color / mirror / Atom feed
* Wrong error message in ia64 assembler
@ 2005-04-18 20:07 Andreas Schwab
  2005-04-18 20:27 ` James E Wilson
  0 siblings, 1 reply; 2+ messages in thread
From: Andreas Schwab @ 2005-04-18 20:07 UTC (permalink / raw)
  To: binutils

$ echo mov.i ar.fpsr=r0 | as -
{standard input}: Assembler messages:
{standard input}:1: Error: AR 40 cannot be accessed by M-unit

This should either say "cannot be accessed by I-unit" or "can only be
access by M-unit".  I chose the latter.  Ok?

Andreas.

2005-04-18  Andreas Schwab  <schwab@suse.de>

	* config/tc-ia64.c (md_assemble): Fix error message about invalid
	access to application register.

--- gas/config/tc-ia64.c.~1.152.~	2005-04-05 11:11:13.000000000 +0200
+++ gas/config/tc-ia64.c	2005-04-18 22:01:00.000000000 +0200
@@ -10694,7 +10694,7 @@ md_assemble (str)
 	  else if (ar_is_only_in_memory_unit (CURR_SLOT.opnd[rop].X_add_number))
 	    unit = 'm';
 	  if (unit != 'a' && unit != idesc->name [4])
-	    as_bad ("AR %d cannot be accessed by %c-unit",
+	    as_bad ("AR %d can only be accessed by %c-unit",
 		    (int) (CURR_SLOT.opnd[rop].X_add_number - REG_AR),
 		    TOUPPER (unit));
 	}

-- 
Andreas Schwab, SuSE Labs, schwab@suse.de
SuSE Linux Products GmbH, Maxfeldstraße 5, 90409 Nürnberg, Germany
Key fingerprint = 58CA 54C7 6D53 942B 1756  01D3 44D5 214B 8276 4ED5
"And now for something completely different."

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

* Re: Wrong error message in ia64 assembler
  2005-04-18 20:07 Wrong error message in ia64 assembler Andreas Schwab
@ 2005-04-18 20:27 ` James E Wilson
  0 siblings, 0 replies; 2+ messages in thread
From: James E Wilson @ 2005-04-18 20:27 UTC (permalink / raw)
  To: Andreas Schwab; +Cc: binutils

On Mon, 2005-04-18 at 13:07, Andreas Schwab wrote:
> 	* config/tc-ia64.c (md_assemble): Fix error message about invalid
> 	access to application register.

OK, except that there is a testcase that must also be fixed.
    gas/testsuite/gas/ia64/invalid-ar.l
Just replacing "cannot" with "can only" will work.
-- 
Jim Wilson, GNU Tools Support, http://www.SpecifixInc.com


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

end of thread, other threads:[~2005-04-18 20:27 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2005-04-18 20:07 Wrong error message in ia64 assembler Andreas Schwab
2005-04-18 20:27 ` James E Wilson

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).