Automatic date update in version.in
[deliverable/binutils-gdb.git] / opcodes / i386-dis.c
index 1a5ea0e62a0ba9189cf2d7afad5bd5e0d13bea52..21e40850544405a32a1dea4d1625917a79d281a8 100644 (file)
@@ -345,6 +345,7 @@ fetch_data (struct disassemble_info *info, bfd_byte *addr)
 #define MX { OP_MMX, 0 }
 #define XM { OP_XMM, 0 }
 #define XMScalar { OP_XMM, scalar_mode }
+#define XMGatherD { OP_XMM, vex_vsib_d_w_dq_mode }
 #define XMGatherQ { OP_XMM, vex_vsib_q_w_dq_mode }
 #define XMM { OP_XMM, xmm_mode }
 #define TMM { OP_XMM, tmm_mode }
@@ -390,6 +391,7 @@ fetch_data (struct disassemble_info *info, bfd_byte *addr)
 #define VexW { OP_VexW, vex_mode }
 #define VexScalar { OP_VEX, vex_scalar_mode }
 #define VexScalarR { OP_VexR, vex_scalar_mode }
+#define VexGatherD { OP_VEX, vex_vsib_d_w_dq_mode }
 #define VexGatherQ { OP_VEX, vex_vsib_q_w_dq_mode }
 #define VexGdq { OP_VEX, dq_mode }
 #define VexTmm { OP_VEX, tmm_mode }
@@ -411,9 +413,7 @@ fetch_data (struct disassemble_info *info, bfd_byte *addr)
 #define MaskVex { OP_VEX, mask_mode }
 
 #define MVexVSIBDWpX { OP_M, vex_vsib_d_w_dq_mode }
-#define MVexVSIBDQWpX { OP_M, vex_vsib_d_w_d_mode }
 #define MVexVSIBQWpX { OP_M, vex_vsib_q_w_dq_mode }
-#define MVexVSIBQDWpX { OP_M, vex_vsib_q_w_d_mode }
 
 #define MVexSIBMEM { OP_M, vex_sibmem_mode }
 
@@ -555,12 +555,8 @@ enum
 
   /* Operand size depends on the VEX.W bit, with VSIB dword indices.  */
   vex_vsib_d_w_dq_mode,
-  /* Similar to vex_vsib_d_w_dq_mode, with smaller memory.  */
-  vex_vsib_d_w_d_mode,
   /* Operand size depends on the VEX.W bit, with VSIB qword indices.  */
   vex_vsib_q_w_dq_mode,
-  /* Similar to vex_vsib_q_w_dq_mode, with smaller memory.  */
-  vex_vsib_q_w_d_mode,
   /* mandatory non-vector SIB.  */
   vex_sibmem_mode,
 
@@ -708,22 +704,24 @@ enum
   REG_VEX_0F3849_X86_64_P_0_W_0_M_1,
   REG_VEX_0F38F3_L_0,
 
-  REG_0FXOP_09_01_L_0,
-  REG_0FXOP_09_02_L_0,
-  REG_0FXOP_09_12_M_1_L_0,
-  REG_0FXOP_0A_12_L_0,
+  REG_XOP_09_01_L_0,
+  REG_XOP_09_02_L_0,
+  REG_XOP_09_12_M_1_L_0,
+  REG_XOP_0A_12_L_0,
 
   REG_EVEX_0F71,
   REG_EVEX_0F72,
   REG_EVEX_0F73,
   REG_EVEX_0F38C6_M_0_L_2,
-  REG_EVEX_0F38C7_M_0_L_2_W_0,
-  REG_EVEX_0F38C7_M_0_L_2_W_1
+  REG_EVEX_0F38C7_M_0_L_2
 };
 
 enum
 {
-  MOD_8D = 0,
+  MOD_62_32BIT = 0,
+  MOD_8D,
+  MOD_C4_32BIT,
+  MOD_C5_32BIT,
   MOD_C6_REG_7,
   MOD_C7_REG_7,
   MOD_FF_REG_3,
@@ -791,9 +789,7 @@ enum
   MOD_0F38FA_PREFIX_1,
   MOD_0F38FB_PREFIX_1,
   MOD_0F3A0F_PREFIX_1,
-  MOD_62_32BIT,
-  MOD_C4_32BIT,
-  MOD_C5_32BIT,
+
   MOD_VEX_0F12_PREFIX_0,
   MOD_VEX_0F12_PREFIX_2,
   MOD_VEX_0F13,
@@ -848,7 +844,7 @@ enum
   MOD_VEX_0F3A32_L_0,
   MOD_VEX_0F3A33_L_0,
 
-  MOD_VEX_0FXOP_09_12,
+  MOD_XOP_09_12,
 
   MOD_EVEX_0F12_PREFIX_0,
   MOD_EVEX_0F12_PREFIX_2,
@@ -883,9 +879,10 @@ enum
   RM_0F01_REG_5_MOD_3,
   RM_0F01_REG_7_MOD_3,
   RM_0F1E_P_1_MOD_3_REG_7,
-  RM_0F3A0F_P_1_MOD_3_REG_0,
   RM_0FAE_REG_6_MOD_3_P_0,
   RM_0FAE_REG_7_MOD_3,
+  RM_0F3A0F_P_1_MOD_3_REG_0,
+
   RM_VEX_0F3849_X86_64_P_0_W_0_M_1_R_0
 };
 
@@ -1145,12 +1142,6 @@ enum
   X86_64_0F01_REG_1_RM_7_PREFIX_2,
   X86_64_0F01_REG_2,
   X86_64_0F01_REG_3,
-  X86_64_0F24,
-  X86_64_0F26,
-  X86_64_VEX_0F3849,
-  X86_64_VEX_0F384B,
-  X86_64_VEX_0F385C,
-  X86_64_VEX_0F385E,
   X86_64_0F01_REG_5_MOD_3_RM_4_PREFIX_1,
   X86_64_0F01_REG_5_MOD_3_RM_5_PREFIX_1,
   X86_64_0F01_REG_5_MOD_3_RM_6_PREFIX_1,
@@ -1158,7 +1149,14 @@ enum
   X86_64_0F01_REG_7_MOD_3_RM_6_PREFIX_1,
   X86_64_0F01_REG_7_MOD_3_RM_6_PREFIX_3,
   X86_64_0F01_REG_7_MOD_3_RM_7_PREFIX_1,
-  X86_64_0FC7_REG_6_MOD_3_PREFIX_1
+  X86_64_0F24,
+  X86_64_0F26,
+  X86_64_0FC7_REG_6_MOD_3_PREFIX_1,
+
+  X86_64_VEX_0F3849,
+  X86_64_VEX_0F384B,
+  X86_64_VEX_0F385C,
+  X86_64_VEX_0F385E
 };
 
 enum
@@ -1334,13 +1332,7 @@ enum
 
 enum
 {
-  EVEX_LEN_0F6E = 0,
-  EVEX_LEN_0F7E_P_1,
-  EVEX_LEN_0F7E_P_2,
-  EVEX_LEN_0FC4,
-  EVEX_LEN_0FC5,
-  EVEX_LEN_0FD6,
-  EVEX_LEN_0F3816,
+  EVEX_LEN_0F3816 = 0,
   EVEX_LEN_0F3819,
   EVEX_LEN_0F381A_M_0,
   EVEX_LEN_0F381B_M_0,
@@ -1351,17 +1343,10 @@ enum
   EVEX_LEN_0F38C7_M_0,
   EVEX_LEN_0F3A00,
   EVEX_LEN_0F3A01,
-  EVEX_LEN_0F3A14,
-  EVEX_LEN_0F3A15,
-  EVEX_LEN_0F3A16,
-  EVEX_LEN_0F3A17,
   EVEX_LEN_0F3A18,
   EVEX_LEN_0F3A19,
   EVEX_LEN_0F3A1A,
   EVEX_LEN_0F3A1B,
-  EVEX_LEN_0F3A20,
-  EVEX_LEN_0F3A21_W_0,
-  EVEX_LEN_0F3A22,
   EVEX_LEN_0F3A23,
   EVEX_LEN_0F3A38,
   EVEX_LEN_0F3A39,
@@ -1549,7 +1534,7 @@ enum
   EVEX_W_0FD2,
   EVEX_W_0FD3,
   EVEX_W_0FD4,
-  EVEX_W_0FD6_L_0,
+  EVEX_W_0FD6,
   EVEX_W_0FE6_P_1,
   EVEX_W_0FE6_P_2,
   EVEX_W_0FE6_P_3,
@@ -1608,11 +1593,6 @@ enum
   EVEX_W_0F387A,
   EVEX_W_0F387B,
   EVEX_W_0F3883,
-  EVEX_W_0F3891,
-  EVEX_W_0F3893,
-  EVEX_W_0F38A1,
-  EVEX_W_0F38A3,
-  EVEX_W_0F38C7_M_0_L_2,
 
   EVEX_W_0F3A05,
   EVEX_W_0F3A08,
@@ -2957,7 +2937,7 @@ static const struct dis386 reg_table[][8] = {
     { "blsmskS",       { VexGdq, Edq }, PREFIX_OPCODE },
     { "blsiS",         { VexGdq, Edq }, PREFIX_OPCODE },
   },
-  /* REG_0FXOP_09_01_L_0 */
+  /* REG_XOP_09_01_L_0 */
   {
     { Bad_Opcode },
     { "blcfill",       { VexGdq, Edq }, 0 },
@@ -2968,7 +2948,7 @@ static const struct dis386 reg_table[][8] = {
     { "blsic", { VexGdq, Edq }, 0 },
     { "t1mskc",        { VexGdq, Edq }, 0 },
   },
-  /* REG_0FXOP_09_02_L_0 */
+  /* REG_XOP_09_02_L_0 */
   {
     { Bad_Opcode },
     { "blcmsk",        { VexGdq, Edq }, 0 },
@@ -2978,12 +2958,12 @@ static const struct dis386 reg_table[][8] = {
     { Bad_Opcode },
     { "blci",  { VexGdq, Edq }, 0 },
   },
-  /* REG_0FXOP_09_12_M_1_L_0 */
+  /* REG_XOP_09_12_M_1_L_0 */
   {
     { "llwpcb",        { Edq }, 0 },
     { "slwpcb",        { Edq }, 0 },
   },
-  /* REG_0FXOP_0A_12_L_0 */
+  /* REG_XOP_0A_12_L_0 */
   {
     { "lwpins",        { VexGdq, Ed, Id }, 0 },
     { "lwpval",        { VexGdq, Ed, Id }, 0 },
@@ -4329,86 +4309,86 @@ static const struct dis386 x86_64_table[][2] = {
     { "lidt", { M }, 0 },
   },
 
+  /* X86_64_0F01_REG_5_MOD_3_RM_4_PREFIX_1 */
   {
-    /* X86_64_0F24 */
-    { "movZ",          { Em, Td }, 0 },
+    { Bad_Opcode },
+    { "uiret", { Skip_MODRM }, 0 },
   },
 
+  /* X86_64_0F01_REG_5_MOD_3_RM_5_PREFIX_1 */
   {
-    /* X86_64_0F26 */
-    { "movZ",          { Td, Em }, 0 },
+    { Bad_Opcode },
+    { "testui",        { Skip_MODRM }, 0 },
   },
 
-  /* X86_64_VEX_0F3849 */
+  /* X86_64_0F01_REG_5_MOD_3_RM_6_PREFIX_1 */
   {
     { Bad_Opcode },
-    { PREFIX_TABLE (PREFIX_VEX_0F3849_X86_64) },
+    { "clui",  { Skip_MODRM }, 0 },
   },
 
-  /* X86_64_VEX_0F384B */
+  /* X86_64_0F01_REG_5_MOD_3_RM_7_PREFIX_1 */
   {
     { Bad_Opcode },
-    { PREFIX_TABLE (PREFIX_VEX_0F384B_X86_64) },
+    { "stui",  { Skip_MODRM }, 0 },
   },
 
-  /* X86_64_VEX_0F385C */
+  /* X86_64_0F01_REG_7_MOD_3_RM_6_PREFIX_1 */
   {
     { Bad_Opcode },
-    { PREFIX_TABLE (PREFIX_VEX_0F385C_X86_64) },
+    { "rmpadjust",     { Skip_MODRM }, 0 },
   },
 
-  /* X86_64_VEX_0F385E */
+  /* X86_64_0F01_REG_7_MOD_3_RM_6_PREFIX_3 */
   {
     { Bad_Opcode },
-    { PREFIX_TABLE (PREFIX_VEX_0F385E_X86_64) },
+    { "rmpupdate",     { Skip_MODRM }, 0 },
   },
 
-  /* X86_64_0F01_REG_5_MOD_3_RM_4_PREFIX_1 */
+  /* X86_64_0F01_REG_7_MOD_3_RM_7_PREFIX_1 */
   {
     { Bad_Opcode },
-    { "uiret", { Skip_MODRM }, 0 },
+    { "psmash",        { Skip_MODRM }, 0 },
   },
 
-  /* X86_64_0F01_REG_5_MOD_3_RM_5_PREFIX_1 */
   {
-    { Bad_Opcode },
-    { "testui",        { Skip_MODRM }, 0 },
+    /* X86_64_0F24 */
+    { "movZ",          { Em, Td }, 0 },
   },
 
-  /* X86_64_0F01_REG_5_MOD_3_RM_6_PREFIX_1 */
   {
-    { Bad_Opcode },
-    { "clui",  { Skip_MODRM }, 0 },
+    /* X86_64_0F26 */
+    { "movZ",          { Td, Em }, 0 },
   },
 
-  /* X86_64_0F01_REG_5_MOD_3_RM_7_PREFIX_1 */
+  /* X86_64_0FC7_REG_6_MOD_3_PREFIX_1 */
   {
     { Bad_Opcode },
-    { "stui",  { Skip_MODRM }, 0 },
+    { "senduipi",      { Eq }, 0 },
   },
 
-  /* X86_64_0F01_REG_7_MOD_3_RM_6_PREFIX_1 */
+  /* X86_64_VEX_0F3849 */
   {
     { Bad_Opcode },
-    { "rmpadjust",     { Skip_MODRM }, 0 },
+    { PREFIX_TABLE (PREFIX_VEX_0F3849_X86_64) },
   },
 
-  /* X86_64_0F01_REG_7_MOD_3_RM_6_PREFIX_3 */
+  /* X86_64_VEX_0F384B */
   {
     { Bad_Opcode },
-    { "rmpupdate",     { Skip_MODRM }, 0 },
+    { PREFIX_TABLE (PREFIX_VEX_0F384B_X86_64) },
   },
 
-  /* X86_64_0F01_REG_7_MOD_3_RM_7_PREFIX_1 */
+  /* X86_64_VEX_0F385C */
   {
     { Bad_Opcode },
-    { "psmash",        { Skip_MODRM }, 0 },
+    { PREFIX_TABLE (PREFIX_VEX_0F385C_X86_64) },
   },
 
-  /* X86_64_0FC7_REG_6_MOD_3_PREFIX_1 */
+  /* X86_64_VEX_0F385E */
   {
     { Bad_Opcode },
-    { "senduipi",      { Eq }, 0 },
+    { PREFIX_TABLE (PREFIX_VEX_0F385E_X86_64) },
   },
 };
 
@@ -5313,7 +5293,7 @@ static const struct dis386 xop_table[][256] = {
     /* 10 */
     { Bad_Opcode },
     { Bad_Opcode },
-    { MOD_TABLE (MOD_VEX_0FXOP_09_12) },
+    { MOD_TABLE (MOD_XOP_09_12) },
     { Bad_Opcode },
     { Bad_Opcode },
     { Bad_Opcode },
@@ -6331,9 +6311,9 @@ static const struct dis386 vex_table[][256] = {
     { MOD_TABLE (MOD_VEX_0F388E) },
     { Bad_Opcode },
     /* 90 */
-    { "vpgatherd%DQ", { XM, MVexVSIBDWpX, Vex }, PREFIX_DATA },
+    { "vpgatherd%DQ", { XM, MVexVSIBDWpX, VexGatherD }, PREFIX_DATA },
     { "vpgatherq%DQ", { XMGatherQ, MVexVSIBQWpX, VexGatherQ }, PREFIX_DATA },
-    { "vgatherdp%XW", { XM, MVexVSIBDWpX, Vex }, PREFIX_DATA },
+    { "vgatherdp%XW", { XM, MVexVSIBDWpX, VexGatherD }, PREFIX_DATA },
     { "vgatherqp%XW", { XMGatherQ, MVexVSIBQWpX, VexGatherQ }, PREFIX_DATA },
     { Bad_Opcode },
     { Bad_Opcode },
@@ -7300,17 +7280,17 @@ static const struct dis386 vex_len_table[][2] = {
 
   /* VEX_LEN_0FXOP_09_01 */
   {
-    { REG_TABLE (REG_0FXOP_09_01_L_0) },
+    { REG_TABLE (REG_XOP_09_01_L_0) },
   },
 
   /* VEX_LEN_0FXOP_09_02 */
   {
-    { REG_TABLE (REG_0FXOP_09_02_L_0) },
+    { REG_TABLE (REG_XOP_09_02_L_0) },
   },
 
   /* VEX_LEN_0FXOP_09_12_M_1 */
   {
-    { REG_TABLE (REG_0FXOP_09_12_M_1_L_0) },
+    { REG_TABLE (REG_XOP_09_12_M_1_L_0) },
   },
 
   /* VEX_LEN_0FXOP_09_82_W_0 */
@@ -7460,7 +7440,7 @@ static const struct dis386 vex_len_table[][2] = {
 
   /* VEX_LEN_0FXOP_0A_12 */
   {
-    { REG_TABLE (REG_0FXOP_0A_12_L_0) },
+    { REG_TABLE (REG_XOP_0A_12_L_0) },
   },
 };
 
@@ -7930,10 +7910,25 @@ static const struct dis386 vex_w_table[][2] = {
 };
 
 static const struct dis386 mod_table[][2] = {
+  {
+    /* MOD_62_32BIT */
+    { "bound{S|}",     { Gv, Ma }, 0 },
+    { EVEX_TABLE (EVEX_0F) },
+  },
   {
     /* MOD_8D */
     { "leaS",          { Gv, M }, 0 },
   },
+  {
+    /* MOD_C4_32BIT */
+    { "lesS",          { Gv, Mp }, 0 },
+    { VEX_C4_TABLE (VEX_0F) },
+  },
+  {
+    /* MOD_C5_32BIT */
+    { "ldsS",          { Gv, Mp }, 0 },
+    { VEX_C5_TABLE (VEX_0F) },
+  },
   {
     /* MOD_C6_REG_7 */
     { Bad_Opcode },
@@ -8240,21 +8235,6 @@ static const struct dis386 mod_table[][2] = {
     { Bad_Opcode },
     { REG_TABLE (REG_0F3A0F_PREFIX_1_MOD_3) },
   },
-  {
-    /* MOD_62_32BIT */
-    { "bound{S|}",     { Gv, Ma }, 0 },
-    { EVEX_TABLE (EVEX_0F) },
-  },
-  {
-    /* MOD_C4_32BIT */
-    { "lesS",          { Gv, Mp }, 0 },
-    { VEX_C4_TABLE (VEX_0F) },
-  },
-  {
-    /* MOD_C5_32BIT */
-    { "ldsS",          { Gv, Mp }, 0 },
-    { VEX_C5_TABLE (VEX_0F) },
-  },
   {
     /* MOD_VEX_0F12_PREFIX_0 */
     { VEX_LEN_TABLE (VEX_LEN_0F12_P_0_M_0) },
@@ -8498,7 +8478,7 @@ static const struct dis386 mod_table[][2] = {
     { "kshiftl%DQ",    { MaskG, MaskE, Ib }, PREFIX_DATA },
   },
   {
-    /* MOD_VEX_0FXOP_09_12 */
+    /* MOD_XOP_09_12 */
     { Bad_Opcode },
     { VEX_LEN_TABLE (VEX_LEN_0FXOP_09_12_M_1) },
   },
@@ -8590,10 +8570,6 @@ static const struct dis386 rm_table[][8] = {
     { "nopQ",          { Ev }, PREFIX_IGNORED },
     { "nopQ",          { Ev }, PREFIX_IGNORED },
   },
-  {
-    /* RM_0F3A0F_P_1_MOD_3_REG_0 */
-    { "hreset",                { Skip_MODRM, Ib }, 0 },
-  },
   {
     /* RM_0FAE_REG_6_MOD_3 */
     { "mfence",                { Skip_MODRM }, 0 },
@@ -8601,7 +8577,10 @@ static const struct dis386 rm_table[][8] = {
   {
     /* RM_0FAE_REG_7_MOD_3 */
     { "sfence",                { Skip_MODRM }, 0 },
-
+  },
+  {
+    /* RM_0F3A0F_P_1_MOD_3_REG_0 */
+    { "hreset",                { Skip_MODRM, Ib }, 0 },
   },
   {
     /* RM_VEX_0F3849_X86_64_P_0_W_0_M_1_R_0 */
@@ -8620,7 +8599,7 @@ static const struct dis386 rm_table[][8] = {
 #define NOTRACK_PREFIX (0x3e | 0x100)
 
 /* Remember if the current op is a jump instruction.  */
-static bfd_boolean op_is_jump = FALSE;
+static bool op_is_jump = false;
 
 static int
 ckprefix (void)
@@ -9063,9 +9042,16 @@ get_valid_dis386 (const struct dis386 *dp, disassemble_info *info)
        case 128:
          vindex = 0;
          break;
+       case 512:
+         /* This allows re-using in particular table entries where only
+            128-bit operand size (VEX.L=0 / EVEX.L'L=0) are valid.  */
+         if (vex.evex)
+           {
        case 256:
-         vindex = 1;
-         break;
+             vindex = 1;
+             break;
+           }
+       /* Fall through.  */
        default:
          abort ();
          break;
@@ -9429,38 +9415,38 @@ print_insn (bfd_vma pc, disassemble_info *info)
 
   for (p = info->disassembler_options; p != NULL; )
     {
-      if (CONST_STRNEQ (p, "amd64"))
+      if (startswith (p, "amd64"))
        isa64 = amd64;
-      else if (CONST_STRNEQ (p, "intel64"))
+      else if (startswith (p, "intel64"))
        isa64 = intel64;
-      else if (CONST_STRNEQ (p, "x86-64"))
+      else if (startswith (p, "x86-64"))
        {
          address_mode = mode_64bit;
          priv.orig_sizeflag |= AFLAG | DFLAG;
        }
-      else if (CONST_STRNEQ (p, "i386"))
+      else if (startswith (p, "i386"))
        {
          address_mode = mode_32bit;
          priv.orig_sizeflag |= AFLAG | DFLAG;
        }
-      else if (CONST_STRNEQ (p, "i8086"))
+      else if (startswith (p, "i8086"))
        {
          address_mode = mode_16bit;
          priv.orig_sizeflag &= ~(AFLAG | DFLAG);
        }
-      else if (CONST_STRNEQ (p, "intel"))
+      else if (startswith (p, "intel"))
        {
          intel_syntax = 1;
-         if (CONST_STRNEQ (p + 5, "-mnemonic"))
+         if (startswith (p + 5, "-mnemonic"))
            intel_mnemonic = 1;
        }
-      else if (CONST_STRNEQ (p, "att"))
+      else if (startswith (p, "att"))
        {
          intel_syntax = 0;
-         if (CONST_STRNEQ (p + 3, "-mnemonic"))
+         if (startswith (p + 3, "-mnemonic"))
            intel_mnemonic = 0;
        }
-      else if (CONST_STRNEQ (p, "addr"))
+      else if (startswith (p, "addr"))
        {
          if (address_mode == mode_64bit)
            {
@@ -9477,14 +9463,14 @@ print_insn (bfd_vma pc, disassemble_info *info)
                priv.orig_sizeflag |= AFLAG;
            }
        }
-      else if (CONST_STRNEQ (p, "data"))
+      else if (startswith (p, "data"))
        {
          if (p[4] == '1' && p[5] == '6')
            priv.orig_sizeflag &= ~DFLAG;
          else if (p[4] == '3' && p[5] == '2')
            priv.orig_sizeflag |= DFLAG;
        }
-      else if (CONST_STRNEQ (p, "suffix"))
+      else if (startswith (p, "suffix"))
        priv.orig_sizeflag |= SUFFIX_ALWAYS;
 
       p = strchr (p, ',');
@@ -9698,6 +9684,13 @@ print_insn (bfd_vma pc, disassemble_info *info)
                    }
                  if (vex.zeroing)
                    oappend ("{z}");
+
+                 /* S/G insns require a mask and don't allow
+                    zeroing-masking.  */
+                 if ((dp->op[0].bytemode == vex_vsib_d_w_dq_mode
+                      || dp->op[0].bytemode == vex_vsib_q_w_dq_mode)
+                     && (vex.mask_register_specifier == 0 || vex.zeroing))
+                   oappend ("/(bad)");
                }
            }
        }
@@ -9715,7 +9708,7 @@ print_insn (bfd_vma pc, disassemble_info *info)
     }
 
   /* Reset jump operation indicator.  */
-  op_is_jump = FALSE;
+  op_is_jump = false;
 
   {
     int jump_detection = 0;
@@ -9737,7 +9730,7 @@ print_insn (bfd_vma pc, disassemble_info *info)
     /* Determine if this is a jump or branch.  */
     if ((jump_detection & 0x3) == 0x3)
       {
-       op_is_jump = TRUE;
+       op_is_jump = true;
        if (jump_detection & 0x4)
          the_info->insn_type = dis_condbranch;
        else
@@ -9755,6 +9748,13 @@ print_insn (bfd_vma pc, disassemble_info *info)
       return end_codep - priv.the_buffer;
     }
 
+  /* If EVEX.z is set, there must be an actual mask register in use.  */
+  if (vex.zeroing && vex.mask_register_specifier == 0)
+    {
+      (*info->fprintf_func) (info->stream, "(bad)");
+      return end_codep - priv.the_buffer;
+    }
+
   switch (dp->prefix_requirement)
     {
     case PREFIX_DATA:
@@ -9799,7 +9799,7 @@ print_insn (bfd_vma pc, disassemble_info *info)
         origins in all_prefixes.  */
       used_prefixes &= ~PREFIX_OPCODE;
       if (last_data_prefix >= 0)
-       all_prefixes[last_repz_prefix] = 0x66;
+       all_prefixes[last_data_prefix] = 0x66;
       if (last_repz_prefix >= 0)
        all_prefixes[last_repz_prefix] = 0xf3;
       if (last_repnz_prefix >= 0)
@@ -10765,7 +10765,7 @@ putop (const char *in_template, int sizeflag)
            }
          else if (l == 1 && last[0] == 'X')
            {
-             if (!need_vex || !vex.evex)
+             if (!vex.evex)
                abort ();
              if (intel_syntax
                  || ((modrm.mod == 3 || vex.b) && !(sizeflag & SUFFIX_ALWAYS)))
@@ -10967,8 +10967,7 @@ print_displacement (char *buf, bfd_vma disp)
 static void
 intel_operand_size (int bytemode, int sizeflag)
 {
-  if (vex.evex
-      && vex.b
+  if (vex.b
       && (bytemode == x_mode
          || bytemode == evex_half_bcst_xmmq_mode))
     {
@@ -11267,51 +11266,10 @@ intel_operand_size (int bytemode, int sizeflag)
       if (!need_vex)
        abort ();
 
-      if (!vex.evex)
-       {
-         if (vex.w)
-           oappend ("QWORD PTR ");
-         else
-           oappend ("DWORD PTR ");
-       }
+      if (vex.w)
+       oappend ("QWORD PTR ");
       else
-       {
-         switch (vex.length)
-           {
-           case 128:
-             oappend ("XMMWORD PTR ");
-             break;
-           case 256:
-             oappend ("YMMWORD PTR ");
-             break;
-           case 512:
-             oappend ("ZMMWORD PTR ");
-             break;
-           default:
-             abort ();
-           }
-       }
-      break;
-    case vex_vsib_q_w_d_mode:
-    case vex_vsib_d_w_d_mode:
-      if (!need_vex || !vex.evex)
-       abort ();
-
-      switch (vex.length)
-       {
-       case 128:
-         oappend ("QWORD PTR ");
-         break;
-       case 256:
-         oappend ("XMMWORD PTR ");
-         break;
-       case 512:
-         oappend ("YMMWORD PTR ");
-         break;
-       default:
-         abort ();
-       }
-
+       oappend ("DWORD PTR ");
       break;
     case mask_bd_mode:
       if (!need_vex || vex.length != 128)
@@ -11475,7 +11433,6 @@ OP_E_memory (int bytemode, int sizeflag)
       /* In EVEX, if operand doesn't allow broadcast, vex.b should be 0.  */
       if (vex.b
          && bytemode != x_mode
-         && bytemode != xmmq_mode
          && bytemode != evex_half_bcst_xmmq_mode)
        {
          BadOp ();
@@ -11506,15 +11463,12 @@ OP_E_memory (int bytemode, int sizeflag)
            /* fall through */
        case vex_scalar_w_dq_mode:
        case vex_vsib_d_w_dq_mode:
-       case vex_vsib_d_w_d_mode:
        case vex_vsib_q_w_dq_mode:
-       case vex_vsib_q_w_d_mode:
        case evex_x_gscat_mode:
          shift = vex.w ? 3 : 2;
          break;
        case x_mode:
        case evex_half_bcst_xmmq_mode:
-       case xmmq_mode:
          if (vex.b)
            {
              shift = vex.w ? 3 : 2;
@@ -11523,6 +11477,7 @@ OP_E_memory (int bytemode, int sizeflag)
          /* Fall through.  */
        case xmmqd_mode:
        case xmmdw_mode:
+       case xmmq_mode:
        case ymmq_mode:
        case evex_x_nobcst_mode:
        case x_swap_mode:
@@ -11593,6 +11548,7 @@ OP_E_memory (int bytemode, int sizeflag)
                         || bytemode == v_bndmk_mode
                         || bytemode == bnd_mode
                         || bytemode == bnd_swap_mode);
+      bool check_gather = false;
       const char **indexes64 = names64;
       const char **indexes32 = names32;
 
@@ -11611,15 +11567,14 @@ OP_E_memory (int bytemode, int sizeflag)
          switch (bytemode)
            {
            case vex_vsib_d_w_dq_mode:
-           case vex_vsib_d_w_d_mode:
            case vex_vsib_q_w_dq_mode:
-           case vex_vsib_q_w_d_mode:
              if (!need_vex)
                abort ();
              if (vex.evex)
                {
                  if (!vex.v)
                    vindex += 16;
+                 check_gather = obufp == op_out[1];
                }
 
              haveindex = 1;
@@ -11630,16 +11585,14 @@ OP_E_memory (int bytemode, int sizeflag)
                  break;
                case 256:
                  if (!vex.w
-                     || bytemode == vex_vsib_q_w_dq_mode
-                     || bytemode == vex_vsib_q_w_d_mode)
+                     || bytemode == vex_vsib_q_w_dq_mode)
                    indexes64 = indexes32 = names_ymm;
                  else
                    indexes64 = indexes32 = names_xmm;
                  break;
                case 512:
                  if (!vex.w
-                     || bytemode == vex_vsib_q_w_dq_mode
-                     || bytemode == vex_vsib_q_w_d_mode)
+                     || bytemode == vex_vsib_q_w_dq_mode)
                    indexes64 = indexes32 = names_zmm;
                  else
                    indexes64 = indexes32 = names_ymm;
@@ -11658,8 +11611,10 @@ OP_E_memory (int bytemode, int sizeflag)
        }
       else
        {
-         /* mandatory non-vector SIB must have sib */
-         if (bytemode == vex_sibmem_mode)
+         /* Check for mandatory SIB.  */
+         if (bytemode == vex_vsib_d_w_dq_mode
+             || bytemode == vex_vsib_q_w_dq_mode
+             || bytemode == vex_sibmem_mode)
            {
              oappend ("(bad)");
              return;
@@ -11812,6 +11767,19 @@ OP_E_memory (int bytemode, int sizeflag)
 
          *obufp++ = close_char;
          *obufp = '\0';
+
+         if (check_gather)
+           {
+             /* Both XMM/YMM/ZMM registers must be distinct.  */
+             int modrm_reg = modrm.reg;
+
+             if (rex & REX_R)
+               modrm_reg += 8;
+             if (!vex.r)
+               modrm_reg += 16;
+             if (vindex == modrm_reg)
+               oappend ("/(bad)");
+           }
        }
       else if (intel_syntax)
        {
@@ -11830,7 +11798,9 @@ OP_E_memory (int bytemode, int sizeflag)
   else if (bytemode == v_bnd_mode
           || bytemode == v_bndmk_mode
           || bytemode == bnd_mode
-          || bytemode == bnd_swap_mode)
+          || bytemode == bnd_swap_mode
+          || bytemode == vex_vsib_d_w_dq_mode
+          || bytemode == vex_vsib_q_w_dq_mode)
     {
       oappend ("(bad)");
       return;
@@ -11909,13 +11879,11 @@ OP_E_memory (int bytemode, int sizeflag)
          oappend (scratchbuf);
        }
     }
-  if (vex.evex && vex.b
+  if (vex.b
       && (bytemode == x_mode
-         || bytemode == xmmq_mode
          || bytemode == evex_half_bcst_xmmq_mode))
     {
       if (vex.w
-         || bytemode == xmmq_mode
          || bytemode == evex_half_bcst_xmmq_mode)
        {
          switch (vex.length)
@@ -12665,36 +12633,8 @@ OP_XMM (int bytemode, int sizeflag ATTRIBUTE_UNUSED)
        reg += 16;
     }
 
-  if (need_vex
-      && bytemode != xmm_mode
-      && bytemode != xmmq_mode
-      && bytemode != evex_half_bcst_xmmq_mode
-      && bytemode != ymm_mode
-      && bytemode != tmm_mode
-      && bytemode != scalar_mode)
-    {
-      switch (vex.length)
-       {
-       case 128:
-         names = names_xmm;
-         break;
-       case 256:
-         if (vex.w
-             || (bytemode != vex_vsib_q_w_dq_mode
-                 && bytemode != vex_vsib_q_w_d_mode))
-           names = names_ymm;
-         else
-           names = names_xmm;
-         break;
-       case 512:
-         names = names_zmm;
-         break;
-       default:
-         abort ();
-       }
-    }
-  else if (bytemode == xmmq_mode
-          || bytemode == evex_half_bcst_xmmq_mode)
+  if (bytemode == xmmq_mode
+      || bytemode == evex_half_bcst_xmmq_mode)
     {
       switch (vex.length)
        {
@@ -12709,6 +12649,8 @@ OP_XMM (int bytemode, int sizeflag ATTRIBUTE_UNUSED)
          abort ();
        }
     }
+  else if (bytemode == ymm_mode)
+    names = names_ymm;
   else if (bytemode == tmm_mode)
     {
       modrm.reg = reg;
@@ -12719,8 +12661,33 @@ OP_XMM (int bytemode, int sizeflag ATTRIBUTE_UNUSED)
        }
       names = names_tmm;
     }
-  else if (bytemode == ymm_mode)
-    names = names_ymm;
+  else if (need_vex
+          && bytemode != xmm_mode
+          && bytemode != scalar_mode)
+    {
+      switch (vex.length)
+       {
+       case 128:
+         names = names_xmm;
+         break;
+       case 256:
+         if (vex.w
+             || bytemode != vex_vsib_q_w_dq_mode)
+           names = names_ymm;
+         else
+           names = names_xmm;
+         break;
+       case 512:
+         if (vex.w
+             || bytemode != vex_vsib_q_w_dq_mode)
+           names = names_zmm;
+         else
+           names = names_ymm;
+         break;
+       default:
+         abort ();
+       }
+    }
   else
     names = names_xmm;
   oappend (names[reg]);
@@ -13369,7 +13336,7 @@ FXSAVE_Fixup (int bytemode, int sizeflag)
 static void
 OP_VEX (int bytemode, int sizeflag ATTRIBUTE_UNUSED)
 {
-  int reg;
+  int reg, modrm_reg, sib_index = -1;
   const char **names;
 
   if (!need_vex)
@@ -13382,14 +13349,46 @@ OP_VEX (int bytemode, int sizeflag ATTRIBUTE_UNUSED)
   else if (vex.evex && !vex.v)
     reg += 16;
 
-  if (bytemode == vex_scalar_mode)
+  switch (bytemode)
     {
+    case vex_scalar_mode:
       oappend (names_xmm[reg]);
       return;
-    }
 
-  if (bytemode == tmm_mode)
-    {
+    case vex_vsib_d_w_dq_mode:
+    case vex_vsib_q_w_dq_mode:
+      /* This must be the 3rd operand.  */
+      if (obufp != op_out[2])
+       abort ();
+      if (vex.length == 128
+         || (bytemode != vex_vsib_d_w_dq_mode
+             && !vex.w))
+       oappend (names_xmm[reg]);
+      else
+       oappend (names_ymm[reg]);
+
+      /* All 3 XMM/YMM registers must be distinct.  */
+      modrm_reg = modrm.reg;
+      if (rex & REX_R)
+       modrm_reg += 8;
+
+      if (modrm.rm == 4)
+       {
+         sib_index = sib.index;
+         if (rex & REX_X)
+           sib_index += 8;
+       }
+
+      if (reg == modrm_reg || reg == sib_index)
+       strcpy (obufp, "/(bad)");
+      if (modrm_reg == sib_index || modrm_reg == reg)
+       strcat (op_out[0], "/(bad)");
+      if (sib_index == modrm_reg || sib_index == reg)
+       strcat (op_out[1], "/(bad)");
+
+      return;
+
+    case tmm_mode:
       /* All 3 TMM registers must be distinct.  */
       if (reg >= 8)
        oappend ("(bad)");
@@ -13422,8 +13421,6 @@ OP_VEX (int bytemode, int sizeflag ATTRIBUTE_UNUSED)
       switch (bytemode)
        {
        case vex_mode:
-       case vex_vsib_q_w_dq_mode:
-       case vex_vsib_q_w_d_mode:
          names = names_xmm;
          break;
        case dq_mode:
@@ -13452,10 +13449,6 @@ OP_VEX (int bytemode, int sizeflag ATTRIBUTE_UNUSED)
        case vex_mode:
          names = names_ymm;
          break;
-       case vex_vsib_q_w_dq_mode:
-       case vex_vsib_q_w_d_mode:
-         names = vex.w ? names_ymm : names_xmm;
-         break;
        case mask_bd_mode:
        case mask_mode:
          if (reg > 0x7)
This page took 0.037908 seconds and 4 git commands to generate.