[rfc]: Remove unnecessary call to gdbarch_bfd_arch_info

[rfc]: Remove unnecessary call to gdbarch_bfd_arch_info

[Markus Deuling]

[-- Attachment #1: Type: text/plain, Size: 352 bytes --]

Hi,

this patch removes a call to gdbarch_bfd_arch_info which is unnecessary, as 
bfd_mach value was set before in every case. Testsuite showed no regressions on x86.

Ok ?

ChangeLog:

	* i386-tdep.c (i386_print_insn): Remove unnecessary call to
	gdbarch_bfd_arch_info.

-- 
  Markus Deuling
  GNU Toolchain for Linux on Cell BE
  deuling@de.ibm.com


[-- Attachment #2: diff-i386-dis --]
[-- Type: text/plain, Size: 530 bytes --]

diff -urpN src/gdb/i386-tdep.c dev/gdb/i386-tdep.c
--- src/gdb/i386-tdep.c	2008-03-11 06:21:52.000000000 +0100
+++ dev/gdb/i386-tdep.c	2008-03-13 09:13:35.000000000 +0100
@@ -2138,7 +2138,6 @@ i386_print_insn (bfd_vma pc, struct disa
   /* FIXME: kettenis/20020915: Until disassembler_options is properly
      constified, cast to prevent a compiler warning.  */
   info->disassembler_options = (char *) disassembly_flavor;
-  info->mach = gdbarch_bfd_arch_info (current_gdbarch)->mach;
 
   return print_insn_i386 (pc, info);
 }

Re: [rfc]: Remove unnecessary call to gdbarch_bfd_arch_info

[Ulrich Weigand]

Markus Deuling wrote:

> 	* i386-tdep.c (i386_print_insn): Remove unnecessary call to
> 	gdbarch_bfd_arch_info.

This looks good to me.

Thanks,
Ulrich

-- 
  Dr. Ulrich Weigand
  GNU Toolchain for Linux on System z and Cell BE
  Ulrich.Weigand@de.ibm.com

Re: [rfc]: Remove unnecessary call to gdbarch_bfd_arch_info

[Markus Deuling]

Ulrich Weigand schrieb:
> Markus Deuling wrote:
> 
>> 	* i386-tdep.c (i386_print_insn): Remove unnecessary call to
>> 	gdbarch_bfd_arch_info.
> 
> This looks good to me.
> 
Thank you, I've committed it.

Regards,
Markus

-- 
  Markus Deuling
  GNU Toolchain for Linux on Cell BE
  deuling@de.ibm.com