[PATCH 2/2] x86: fold conditionals in check_long_reg()
Jan Beulich
jbeulich@suse.com
Mon Nov 6 12:14:13 GMT 2023
Simplify the code follow ing what check_{,q}word_reg() already do. This
the also eliminates a stale comment talking about a warning when an
error is raised. While there, correct a similarly stale comment in
check_qword_reg() while there.
--- a/gas/config/tc-i386.c
+++ b/gas/config/tc-i386.c
@@ -8121,8 +8121,10 @@ check_long_reg (void)
i.suffix);
return 0;
}
- /* Error if the e prefix on a general reg is missing. */
- else if (i.types[op].bitfield.word
+ /* Error if the e prefix on a general reg is missing, or if the r
+ prefix on a general reg is present. */
+ else if ((i.types[op].bitfield.word
+ || i.types[op].bitfield.qword)
&& (i.tm.operand_types[op].bitfield.class == Reg
|| i.tm.operand_types[op].bitfield.instance == Accum)
&& i.tm.operand_types[op].bitfield.dword)
@@ -8132,16 +8134,6 @@ check_long_reg (void)
i.suffix);
return 0;
}
- /* Warn if the r prefix on a general reg is present. */
- else if (i.types[op].bitfield.qword
- && (i.tm.operand_types[op].bitfield.class == Reg
- || i.tm.operand_types[op].bitfield.instance == Accum)
- && i.tm.operand_types[op].bitfield.dword)
- {
- as_bad (_("incorrect register `%s%s' used with `%c' suffix"),
- register_prefix, i.op[op].regs->reg_name, i.suffix);
- return 0;
- }
return 1;
}
@@ -8170,7 +8162,7 @@ check_qword_reg (void)
i.suffix);
return 0;
}
- /* Warn if the r prefix on a general reg is missing. */
+ /* Error if the r prefix on a general reg is missing. */
else if ((i.types[op].bitfield.word
|| i.types[op].bitfield.dword)
&& (i.tm.operand_types[op].bitfield.class == Reg
More information about the Binutils
mailing list