From: Jan Beulich <jbeulich@novell.com>
Date: Tue, 31 Jul 2012 07:45:48 +0000 (+0000)
Subject: The current error message for bad imm4 operands wasn't really helpful,
X-Git-Tag: sid-snapshot-20120801~14
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=4a2608e354d22289f40944bb29d85db3f3c5ca04;p=platform%2Fupstream%2Fbinutils.git

The current error message for bad imm4 operands wasn't really helpful,
and was pointing at the wrong operand in Intel mode. Since non-constant
operands are being taken care of by other means anyway, adjust it to
simply state that the constant doesn't fit.

2012-07-31  Jan Beulich <jbeulich@suse.com>

	* config/tc-i386.c (match_template): Adjust error message
	for 'bad_imm4' case.
---

diff --git a/gas/ChangeLog b/gas/ChangeLog
index d474b3c..be74ff1 100644
--- a/gas/ChangeLog
+++ b/gas/ChangeLog
@@ -1,5 +1,10 @@
 2012-07-31  Jan Beulich <jbeulich@suse.com>
 
+	* config/tc-i386.c (match_template): Adjust error message
+	for 'bad_imm4' case.
+
+2012-07-31  Jan Beulich <jbeulich@suse.com>
+
 	* config/tc-i386.c (check_byte_reg): Check for I/O port
 	register earlier, and just once. Drop diagnostic that got
 	issued only for some registers.
diff --git a/gas/config/tc-i386.c b/gas/config/tc-i386.c
index 70f21db..5303f63d 100644
--- a/gas/config/tc-i386.c
+++ b/gas/config/tc-i386.c
@@ -4347,7 +4347,7 @@ check_reverse:
 	  err_msg = _("invalid instruction suffix");
 	  break;
 	case bad_imm4:
-	  err_msg = _("Imm4 isn't the first operand");
+	  err_msg = _("constant doesn't fit in 4 bits");
 	  break;
 	case old_gcc_only:
 	  err_msg = _("only supported with old gcc");