From: Jan Beulich <jbeulich@suse.com>
Date: Mon, 17 Jan 2022 09:26:21 +0000 (+0100)
Subject: x86: drop index16 field
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=71b94817f314db6ddb428e804378a30e7521a2ad;p=binutils-gdb.git

x86: drop index16 field

There's a single use on a generally infrequently taken code path. Put
the necessary conditional there instead.
---

diff --git a/opcodes/i386-dis.c b/opcodes/i386-dis.c
index c2924ef27e6..cc24d971e73 100644
--- a/opcodes/i386-dis.c
+++ b/opcodes/i386-dis.c
@@ -219,8 +219,6 @@ struct instr_info
   vex;
   unsigned char need_vex;
 
-  const char *const *index16;
-
   /* Remember if the current op is a jump instruction.  */
   bool op_is_jump;
 
@@ -9404,7 +9402,6 @@ print_insn (bfd_vma pc, instr_info *ins)
 
   if (ins->intel_syntax)
     {
-      ins->index16 = intel_index16;
       ins->open_char = '[';
       ins->close_char = ']';
       ins->separator_char = '+';
@@ -9412,7 +9409,6 @@ print_insn (bfd_vma pc, instr_info *ins)
     }
   else
     {
-      ins->index16 = att_index16;
       ins->open_char = '(';
       ins->close_char =  ')';
       ins->separator_char = ',';
@@ -11712,7 +11708,9 @@ OP_E_memory (instr_info *ins, int bytemode, int sizeflag)
 	{
 	  *ins->obufp++ = ins->open_char;
 	  *ins->obufp = '\0';
-	  oappend (ins, ins->index16[ins->modrm.rm]);
+	  oappend (ins,
+		   (ins->intel_syntax ? intel_index16
+				      : att_index16)[ins->modrm.rm]);
 	  if (ins->intel_syntax
 	      && (disp || ins->modrm.mod != 0 || ins->modrm.rm == 6))
 	    {