x86/Intel: drop pointless special casing of LxS
authorJan Beulich <jbeulich@suse.com>
Mon, 9 Dec 2019 12:28:43 +0000 (13:28 +0100)
committerJan Beulich <jbeulich@suse.com>
Mon, 9 Dec 2019 12:28:43 +0000 (13:28 +0100)
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").

gas/ChangeLog
gas/config/tc-i386-intel.c

index 0e9a585432de421258881554945ef7c6fbeeb24e..96bab9654fb88b5e52adfde84a241162ae2ef71b 100644 (file)
@@ -1,3 +1,8 @@
+2019-12-09  Jan Beulich  <jbeulich@suse.com>
+
+       * 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  <amodra@gmail.com>
 
        * testsuite/gas/aarch64/bfloat16.d: Match 32-bit and 64-bit output.
index b639ab7dd6b6743b0e6aa7df75030040b11bd0f6..ff5f0c612dff0fcbf75e771097d7e837e9e37d0f 100644 (file)
@@ -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;