Re: [PATCH, i386] Update to match latest AVX512 spec.

["H.J. Lu" <hjl dot tools at gmail dot com>]

On Wed, Feb 19, 2014 at 9:16 AM, Ilya Tocar <tocarip.intel@gmail.com> wrote:

> Attached is gzipped patch. Testing passes. There are a few trailing whitespaces,
> but only in auto-generated files.

I checked in this patch to avoid trailing space in generated files.

-- 
H.J.
--
diff --git a/opcodes/ChangeLog b/opcodes/ChangeLog
index f861eca..df34675 100644
--- a/opcodes/ChangeLog
+++ b/opcodes/ChangeLog
@@ -1,3 +1,11 @@
+2014-02-19  H.J. Lu  <hongjiu.lu@intel.com>
+
+ * i386-gen.c (output_cpu_flags): Don't output trailing space.
+ (output_opcode_modifier): Likewise.
+ (output_operand_type): Likewise.
+ * i386-init.h: Regenerated.
+ * i386-tbl.h: Likewise.
+
 2014-02-12  Ilya Tocar  <ilya.tocar@intel.com>

  * i386-dis.c (MOD enum): Add MOD_0FC7_REG_3, MOD_0FC7_REG_4,
diff --git a/opcodes/i386-gen.c b/opcodes/i386-gen.c
index 343f58d..59f7e09 100644
--- a/opcodes/i386-gen.c
+++ b/opcodes/i386-gen.c
@@ -713,7 +713,10 @@ output_cpu_flags (FILE *table, bitfield *flags,
unsigned int size,

   for (i = 0; i < size - 1; i++)
     {
-      fprintf (table, "%d, ", flags[i].value);
+      if (((i + 1) % 20) != 0)
+ fprintf (table, "%d, ", flags[i].value);
+      else
+ fprintf (table, "%d,", flags[i].value);
       if (((i + 1) % 20) == 0)
  {
   /* We need \\ for macro.  */
@@ -801,7 +804,10 @@ output_opcode_modifier (FILE *table, bitfield
*modifier, unsigned int size)

   for (i = 0; i < size - 1; i++)
     {
-      fprintf (table, "%d, ", modifier[i].value);
+      if (((i + 1) % 20) != 0)
+        fprintf (table, "%d, ", modifier[i].value);
+      else
+        fprintf (table, "%d,", modifier[i].value);
       if (((i + 1) % 20) == 0)
  fprintf (table, "\n      ");
     }
@@ -842,12 +848,15 @@ output_operand_type (FILE *table, bitfield
*types, unsigned int size,

   for (i = 0; i < size - 1; i++)
     {
-      fprintf (table, "%d, ", types[i].value);
+      if (((i + 1) % 20) != 0)
+ fprintf (table, "%d, ", types[i].value);
+      else
+ fprintf (table, "%d,", types[i].value);
       if (((i + 1) % 20) == 0)
  {
   /* We need \\ for macro.  */
   if (macro)
-    fprintf (table, "\\\n%s", indent);
+    fprintf (table, " \\\n%s", indent);
   else
     fprintf (table, "\n%s", indent);
  }