x86: drop dead SSE2AVX-related code
authorJan Beulich <jbeulich@suse.com>
Fri, 27 Jan 2023 08:23:12 +0000 (09:23 +0100)
committerJan Beulich <jbeulich@suse.com>
Fri, 27 Jan 2023 08:23:12 +0000 (09:23 +0100)
All (there are just two) SSE2AVX templates with %xmm0 as first operand
also specify VEX3SOURCES. Hence there's no need for an "else" to the
respective if(), and the if() itself can become an assertion.

gas/config/tc-i386.c

index be691403984c76791b4121bd07d92304104dcbb7..985984c71eb0a03eafba20d2ac68f4e41e810d2c 100644 (file)
@@ -7965,29 +7965,15 @@ process_operands (void)
       if (i.tm.operand_types[0].bitfield.instance == Accum
          && i.tm.operand_types[0].bitfield.xmmword)
        {
-         if (i.tm.opcode_modifier.vexsources == VEX3SOURCES)
-           {
-             /* Keep xmm0 for instructions with VEX prefix and 3
-                sources.  */
-             i.tm.operand_types[0].bitfield.instance = InstanceNone;
-             i.tm.operand_types[0].bitfield.class = RegSIMD;
-             goto duplicate;
-           }
-         else
-           {
-             /* We remove the first xmm0 and keep the number of
-                operands unchanged, which in fact duplicates the
-                destination.  */
-             for (j = 1; j < i.operands; j++)
-               {
-                 i.op[j - 1] = i.op[j];
-                 i.types[j - 1] = i.types[j];
-                 i.tm.operand_types[j - 1] = i.tm.operand_types[j];
-                 i.flags[j - 1] = i.flags[j];
-               }
-           }
+         gas_assert (i.tm.opcode_modifier.vexsources == VEX3SOURCES);
+         /* Keep xmm0 for instructions with VEX prefix and 3
+            sources.  */
+         i.tm.operand_types[0].bitfield.instance = InstanceNone;
+         i.tm.operand_types[0].bitfield.class = RegSIMD;
+         goto duplicate;
        }
-      else if (i.tm.opcode_modifier.operandconstraint == IMPLICIT_1ST_XMM0)
+
+      if (i.tm.opcode_modifier.operandconstraint == IMPLICIT_1ST_XMM0)
        {
          gas_assert ((MAX_OPERANDS - 1) > dupl
                      && (i.tm.opcode_modifier.vexsources