From: Jan Beulich Date: Mon, 9 Dec 2019 12:28:43 +0000 (+0100) Subject: x86/Intel: drop pointless special casing of LxS X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=37404387124c7abb78c3e21193572e4035f0beb5;p=binutils-gdb.git x86/Intel: drop pointless special casing of LxS LDS et al don't accept "word ptr" operands anyway, as per their insn templates. Hence there's no need to special case this here; the check has become dysfunctional anyway by dc2be329b950 ("i386: Only check suffix in instruction mnemonic"). --- diff --git a/gas/ChangeLog b/gas/ChangeLog index 0e9a585432d..96bab9654fb 100644 --- a/gas/ChangeLog +++ b/gas/ChangeLog @@ -1,3 +1,8 @@ +2019-12-09 Jan Beulich + + * config/tc-i386-intel.c (i386_intel_operand): Don't special + case LDS et al when handling O_word_ptr. + 2019-12-08 Alan Modra * testsuite/gas/aarch64/bfloat16.d: Match 32-bit and 64-bit output. diff --git a/gas/config/tc-i386-intel.c b/gas/config/tc-i386-intel.c index b639ab7dd6b..ff5f0c612df 100644 --- a/gas/config/tc-i386-intel.c +++ b/gas/config/tc-i386-intel.c @@ -639,12 +639,7 @@ i386_intel_operand (char *operand_string, int got_a_float) case O_word_ptr: i.types[this_operand].bitfield.word = 1; - if ((current_templates->start->name[0] == 'l' - && current_templates->start->name[2] == 's' - && current_templates->start->name[3] == 0) - || current_templates->start->base_opcode == 0x62 /* bound */) - suffix = BYTE_MNEM_SUFFIX; /* so it will cause an error */ - else if (got_a_float == 2) /* "fi..." */ + if (got_a_float == 2) /* "fi..." */ suffix = SHORT_MNEM_SUFFIX; else suffix = WORD_MNEM_SUFFIX;