PATCH: Remove no_xsuf

H.J. Lu hjl@lucon.org
Fri Jan 4 17:22:00 GMT 2008


We can use the xmmword field instead of no_xsuf. I am checking in
this patch to remove it.

H.J.
--
gas/

2008-01-03  H.J. Lu  <hongjiu.lu@intel.com>

	* config/tc-i386.c (match_template): Use the xmmword field
	instead of no_xsuf.

opcodes/

2008-01-03  H.J. Lu  <hongjiu.lu@intel.com>

	* i386-gen.c (opcode_modifiers): Remove No_xSuf.

	* i386-opc.h (No_xSuf): Removed.
	(CheckSize): Updated.

	* i386-tbl.h: Regenerated.

--- binutils/gas/config/tc-i386.c.bar	2008-01-02 17:41:51.000000000 -0800
+++ binutils/gas/config/tc-i386.c	2008-01-03 11:58:24.000000000 -0800
@@ -3048,7 +3048,7 @@ match_template (void)
   else if (i.suffix == LONG_DOUBLE_MNEM_SUFFIX)
     suffix_check.no_ldsuf = 1;
   else if (i.suffix == XMMWORD_MNEM_SUFFIX)
-    suffix_check.no_xsuf = 1;
+    suffix_check.xmmword = 1;
 
   for (t = current_templates->start; t < current_templates->end; t++)
     {
@@ -3088,7 +3088,7 @@ match_template (void)
 	  && (!t->opcode_modifier.word || !suffix_check.no_wsuf)
 	  && (!t->opcode_modifier.dword || !suffix_check.no_lsuf)
 	  && (!t->opcode_modifier.qword || !suffix_check.no_qsuf)
-	  && (!t->opcode_modifier.xmmword || !suffix_check.no_xsuf))
+	  && (!t->opcode_modifier.xmmword || !suffix_check.xmmword))
 	continue;
 
       for (j = 0; j < MAX_OPERANDS; j++)
--- binutils/opcodes/i386-gen.c.bar	2008-01-03 10:57:31.000000000 -0800
+++ binutils/opcodes/i386-gen.c	2008-01-03 11:56:50.000000000 -0800
@@ -275,7 +275,6 @@ static bitfield opcode_modifiers[] =
   BITFIELD (No_sSuf),
   BITFIELD (No_qSuf),
   BITFIELD (No_ldSuf),
-  BITFIELD (No_xSuf),
   BITFIELD (CheckSize),
   BITFIELD (Byte),
   BITFIELD (Word),
--- binutils/opcodes/i386-opc.h.bar	2008-01-03 10:57:31.000000000 -0800
+++ binutils/opcodes/i386-opc.h	2008-01-03 11:56:35.000000000 -0800
@@ -195,10 +195,8 @@ typedef union i386_cpu_flags
 #define No_qSuf			(No_sSuf + 1)
 /* long double suffix on instruction illegal */
 #define No_ldSuf		(No_qSuf + 1)
-/* x suffix on instruction illegal */
-#define No_xSuf			(No_ldSuf + 1)
 /* check memory size on instruction in Intel mode if it is specified.  */
-#define CheckSize		(No_xSuf + 1)
+#define CheckSize		(No_ldSuf + 1)
 /* BYTE memory on instruction */
 #define Byte			(CheckSize + 1)
 /* WORD memory on instruction */
@@ -274,7 +272,6 @@ typedef struct i386_opcode_modifier
   unsigned int no_ssuf:1;
   unsigned int no_qsuf:1;
   unsigned int no_ldsuf:1;
-  unsigned int no_xsuf:1;
   unsigned int checksize:1;
   unsigned int byte:1;
   unsigned int word:1;



More information about the Binutils mailing list