From 9754d1a75059281d9ce9b510d8146c5e0ce2152c Mon Sep 17 00:00:00 2001 From: Jan Beulich Date: Fri, 14 Feb 2025 10:47:50 +0100 Subject: [PATCH] x86: drop redundant i.operands checks from output_disp() The opcode space, major opcode, and - where applicable - opcode extension checks fully qualify the insns we're after; operand matching has been done far earlier, so wrong operand counts cannot occur here. --- gas/config/tc-i386.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/gas/config/tc-i386.c b/gas/config/tc-i386.c index d648158b19c..9cfc4df1e10 100644 --- a/gas/config/tc-i386.c +++ b/gas/config/tc-i386.c @@ -12943,17 +12943,14 @@ output_disp (fragS *insn_start_frag, offsetT insn_start_off) && (i.rm.mode == 2 || (i.rm.mode == 0 && i.rm.regmem == 5)) && ((space == SPACE_BASE - && i.operands == 1 && i.tm.base_opcode == 0xff && (i.rm.reg == 2 || i.rm.reg == 4)) || ((space == SPACE_BASE || space == SPACE_0F38 || space == SPACE_MAP4) - && i.operands == 2 && i.tm.base_opcode == 0x8b) || ((space == SPACE_BASE || space == SPACE_MAP4) - && i.operands >= 2 && (i.tm.base_opcode == 0x85 || (i.tm.base_opcode | (i.operands > 2 ? 0x3a : 0x38)) == 0x3b)))) -- 2.47.3