RFA: Skip ARM ELF Mapping symbols when showing disassembly

Nick Clifton nickc@redhat.com
Thu Nov 13 14:29:00 GMT 2003


Hi Guys,

  I have recently committed a patch to the arm-elf port of GAS which
  causes it to generate special mapping symbols as required by the ARM
  ELF spec.  Unfortunately this now means that when GDB shows a
  disassembly it can select one of the mapping symbols instead of the
  proper function name symbol.

  The patch below is a fix for this problem.  It is not elegant, but
  it does have the advantage of being entirely arm specific.  A
  cleaner patch would require changes to generic code, which I did not
  feel comfortable doing.

  May I apply this patch ?

Cheers
        Nick

2003-11-13  Nick Clifton  <nickc@redhat.com>

	* arm-tdep.c (arm_elf_make_msymbol_special): Intercept arm-elf
	mapping symbols and artificially alter their value so that they
	are not used for disassembly displays.

Index: gdb/arm-tdep.c
===================================================================
RCS file: /cvs/src/src/gdb/arm-tdep.c,v
retrieving revision 1.155
diff -c -3 -p -r1.155 arm-tdep.c
*** gdb/arm-tdep.c	11 Nov 2003 20:04:52 -0000	1.155
--- gdb/arm-tdep.c	13 Nov 2003 14:27:49 -0000
*************** coff_sym_is_thumb (int val)
*** 2675,2680 ****
--- 2675,2692 ----
  static void
  arm_elf_make_msymbol_special(asymbol *sym, struct minimal_symbol *msym)
  {
+   /* FIXME: We want gdb to ignore the ARM ELF mapping symbols when
+      displaying disassembly so we use this horrible hack here to
+      artifically set their address to the highest possible value.
+      This is wrong of course, and it prevents the symbols from being
+      used for their intended purpose - to distinguish between ARM
+      and THUMB code.  So we ought to find a better way to do this.  */
+   if (bfd_asymbol_name (sym)
+       && bfd_asymbol_name (sym)[0] == '$'
+       && bfd_asymbol_name (sym)[1] != 0
+       && bfd_asymbol_name (sym)[2] == 0)
+     SYMBOL_VALUE_ADDRESS(msym) = (CORE_ADDR) 0x7ffffffc;
+       
    /* Thumb symbols are of type STT_LOPROC, (synonymous with
       STT_ARM_TFUNC).  */
    if (ELF_ST_TYPE (((elf_symbol_type *)sym)->internal_elf_sym.st_info)



More information about the Gdb-patches mailing list