x86: drop a few redundant EVEX-related checks
authorJan Beulich <jbeulich@suse.com>
Thu, 11 Mar 2021 15:21:19 +0000 (16:21 +0100)
committerJan Beulich <jbeulich@suse.com>
Thu, 11 Mar 2021 15:21:19 +0000 (16:21 +0100)
vex.b can only be set when vex.evex is also set. Similarly vex.evex can
only be set when need_vex is also set.

opcodes/ChangeLog
opcodes/i386-dis.c

index 54c69d856246ff002d3f99223a542b712862793d..e31a93f7de95b38287facbb9e8a253d5e6e3645e 100644 (file)
@@ -1,3 +1,10 @@
+2021-03-11  Jan Beulich  <jbeulich@suse.com>
+
+       * i386-dis.c (putop): Drop need_vex check when also checking
+       vex.evex.
+       (intel_operand_size, OP_E_memory): Drop vex.evex check when also
+       checking vex.b.
+
 2021-03-11  Jan Beulich  <jbeulich@suse.com>
 
        * i386-dis.c (OP_E_memory): Drop xmmq_mode from broadcast
index eb91c5df4a60a8d02ce7d0fa1f61bb8058489d90..10dbe7fe18301a6a1d5cc891c0ee3c987240148c 100644 (file)
@@ -10749,7 +10749,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)))
@@ -10951,8 +10951,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))
     {
@@ -11845,7 +11844,7 @@ OP_E_memory (int bytemode, int sizeflag)
          oappend (scratchbuf);
        }
     }
-  if (vex.evex && vex.b
+  if (vex.b
       && (bytemode == x_mode
          || bytemode == evex_half_bcst_xmmq_mode))
     {