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 d474b3ce3d846e8139d8383c1c657af617a3d474..be74ff1ee6ee151c82a3ab801bf4c4e92a33fbad 100644 (file)
--- a/gas/ChangeLog
+++ b/gas/ChangeLog
@@ -1,3 +1,8 @@
+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

diff --git a/gas/config/tc-i386.c b/gas/config/tc-i386.c
index 70f21db1dd60c020d42eb1a27048b075e6779c21..5303f63da04c2482fca48fd9f5d275f913e00076 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");