This is the mail archive of the
gdb-patches@sources.redhat.com
mailing list for the GDB project.
Re: [ob] Delete PROLOGUE_FIRSTLINE_OVERLAP
- From: Andrew Cagney <cagney at gnu dot org>
- To: Andrew Cagney <cagney at gnu dot org>
- Cc: gdb-patches at sources dot redhat dot com
- Date: Sat, 22 Nov 2003 11:11:01 -0500
- Subject: Re: [ob] Delete PROLOGUE_FIRSTLINE_OVERLAP
- References: <3FBF8844.10007@gnu.org>
patch ...
2003-11-22 Andrew Cagney <cagney@redhat.com>
* config/powerpc/tm-linux.h (PROLOGUE_FIRSTLINE_OVERLAP): Delete
#if 0'ed macro.
* infrun.c (step_into_function): Delete #ifdef
PROLOGUE_FIRSTLINE_OVERLAP code.
* symtab.c (find_function_start_sal): Ditto.
Index: infrun.c
===================================================================
RCS file: /cvs/src/src/gdb/infrun.c,v
retrieving revision 1.119
diff -u -r1.119 infrun.c
--- infrun.c 14 Nov 2003 21:22:42 -0000 1.119
+++ infrun.c 22 Nov 2003 15:57:29 -0000
@@ -2727,15 +2727,10 @@
/* If the prologue ends in the middle of a source line, continue to
the end of that source line (if it is still within the function).
Otherwise, just go to end of prologue. */
-#ifdef PROLOGUE_FIRSTLINE_OVERLAP
- /* no, don't either. It skips any code that's legitimately on the
- first line. */
-#else
if (ecs->sal.end
&& ecs->sal.pc != ecs->stop_func_start
&& ecs->sal.end < ecs->stop_func_end)
ecs->stop_func_start = ecs->sal.end;
-#endif
if (ecs->stop_func_start == stop_pc)
{
Index: symtab.c
===================================================================
RCS file: /cvs/src/src/gdb/symtab.c,v
retrieving revision 1.122
diff -u -r1.122 symtab.c
--- symtab.c 8 Nov 2003 00:13:03 -0000 1.122
+++ symtab.c 22 Nov 2003 15:57:46 -0000
@@ -2314,10 +2314,6 @@
}
sal = find_pc_sect_line (pc, SYMBOL_BFD_SECTION (sym), 0);
-#ifdef PROLOGUE_FIRSTLINE_OVERLAP
- /* Convex: no need to suppress code on first line, if any */
- sal.pc = pc;
-#else
/* Check if SKIP_PROLOGUE left us in mid-line, and the next
line is still part of the same function. */
if (sal.pc != pc
@@ -2330,7 +2326,6 @@
sal = find_pc_sect_line (pc, SYMBOL_BFD_SECTION (sym), 0);
}
sal.pc = pc;
-#endif
return sal;
}
Index: config/powerpc/tm-linux.h
===================================================================
RCS file: /cvs/src/src/gdb/config/powerpc/tm-linux.h,v
retrieving revision 1.16
diff -u -r1.16 tm-linux.h
--- config/powerpc/tm-linux.h 6 Nov 2003 03:29:49 -0000 1.16
+++ config/powerpc/tm-linux.h 22 Nov 2003 15:57:47 -0000
@@ -43,13 +43,6 @@
#define CANNOT_STORE_REGISTER(regno) ((regno) >= MQ_REGNUM)
#endif
-#if 0 /* If skip_prologue() isn't too greedy, we don't need this */
-/* There is some problem with the debugging symbols generated by the
- compiler such that the debugging symbol for the first line of a
- function overlap with the function prologue. */
-#define PROLOGUE_FIRSTLINE_OVERLAP
-#endif
-
/* N_FUN symbols in shared libaries have 0 for their values and need
to be relocated. */
#define SOFUN_ADDRESS_MAYBE_MISSING