]> git.ipfire.org Git - thirdparty/binutils-gdb.git/commitdiff
x86: drop bogus accumulator check
authorJan Beulich <jbeulich@suse.com>
Fri, 16 May 2025 08:25:38 +0000 (10:25 +0200)
committerJan Beulich <jbeulich@suse.com>
Fri, 16 May 2025 08:25:38 +0000 (10:25 +0200)
Accum is an "instance", not a "class". With present enumerator values of
Reg and Accum, the 2nd check simply did the same as the first. In fact
checking for the accumulator (%rax) isn't necessary here at all, because
there's no case where an individual template would permit alternatively
a memory operand or the (qword) accumulator; only "any GPR" is ever
being paired with "memory".

gas/config/tc-i386.c

index 3e7040a3314241cfd9068d43af9ab0f8d5c3b2da..9671c64217866a49c43a89f9a1b9f1d46e5259f8 100644 (file)
@@ -2518,7 +2518,6 @@ match_operand_size (const insn_template *t, unsigned int wanted,
                   || (intel_syntax
                       && flag_code != CODE_64BIT
                       && (t->operand_types[wanted].bitfield.class == Reg
-                          || t->operand_types[wanted].bitfield.class == Accum
                           || t->opcode_modifier.isstring))))
           || (i.types[given].bitfield.tbyte
               && !t->operand_types[wanted].bitfield.tbyte));