x86: drop redundant VSIB handling code

The vecsib && !base_reg case is already being handled (in a more correct
manner) by earlier code.

diff --git a/gas/ChangeLog b/gas/ChangeLog
index c00280754803983c1639db3bc095551e260c9e8a..d871aa3f9fa605400d4222b4c7b10ef19c73a3f4 100644 (file)
--- a/gas/ChangeLog
+++ b/gas/ChangeLog
@@ -1,3 +1,8 @@
+2017-11-23  Jan Beulich  <jbeulich@suse.com>
+
+       * config/tc-i386.c (build_modrm_byte): Drop VSIB handling from
+       code also setting fake_zero_displacement.
+
 2017-11-23  Jan Beulich  <jbeulich@suse.com>
 
        * testsuite/gas/i386/arch-4.s: Correct ud1 and ud2b. Add ud0.

diff --git a/gas/config/tc-i386.c b/gas/config/tc-i386.c
index 5a7066c1dd63d098b596b85a81b9a760a3bccdd2..e22e74ce4db18560314f79e522a4785215f27e98 100644 (file)
--- a/gas/config/tc-i386.c
+++ b/gas/config/tc-i386.c
@@ -6591,13 +6591,7 @@ build_modrm_byte (void)
            {
              i.rm.mode = 0;
              if (!i.disp_operands)
-               {
-                 fake_zero_displacement = 1;
-                 /* Instructions with VSIB byte need 32bit displacement
-                    if there is no base register.  */
-                 if (i.tm.opcode_modifier.vecsib)
-                   i.types[op].bitfield.disp32 = 1;
-               }
+               fake_zero_displacement = 1;
              if (i.index_reg == 0)
                {
                  gas_assert (!i.tm.opcode_modifier.vecsib);