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 (file)
--- 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..5303f63 100644 (file)
--- 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");