From: John Gilmore Date: Wed, 14 Aug 1991 00:00:25 +0000 (+0000) Subject: * blockframe.c (frameless_look_for_prologue): Speed up by X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=5259796be2dcfdbb88a281cab9030d3f37c3bacb;p=binutils-gdb.git * blockframe.c (frameless_look_for_prologue): Speed up by calling SKIP_PROLOGUE_FRAMELESS_P if it's defined. * tm-sparc.h (SKIP_PROLOGUE, SKIP_PROLOGUE_FRAMELESS_P): Define two variants, one faster. * sparc-tdep.c (sparc_frame_chain): Use target_read_memory and return zero, rather than giving an error. (setup_arbitrary_frame): Initialize the PC in the new frame. (skip_prologue): Add a second arg saying whether we're just checking for a frameless function or not. --- diff --git a/gdb/blockframe.c b/gdb/blockframe.c index ba4cc245336..7efe1411270 100644 --- a/gdb/blockframe.c +++ b/gdb/blockframe.c @@ -4,19 +4,19 @@ This file is part of GDB. -GDB is free software; you can redistribute it and/or modify +This program is free software; you can redistribute it and/or modify it under the terms of the GNU General Public License as published by -the Free Software Foundation; either version 1, or (at your option) -any later version. +the Free Software Foundation; either version 2 of the License, or +(at your option) any later version. -GDB is distributed in the hope that it will be useful, +This program is distributed in the hope that it will be useful, but WITHOUT ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License for more details. You should have received a copy of the GNU General Public License -along with GDB; see the file COPYING. If not, write to -the Free Software Foundation, 675 Mass Ave, Cambridge, MA 02139, USA. */ +along with this program; if not, write to the Free Software +Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA. */ #include #include "defs.h" @@ -191,7 +191,13 @@ frameless_look_for_prologue (frame) if (func_start) { after_prologue = func_start; +#ifdef SKIP_PROLOGUE_FRAMELESS_P + /* This is faster, since only care whether there *is* a prologue, + not how long it is. */ + SKIP_PROLOGUE_FRAMELESS_P (after_prologue); +#else SKIP_PROLOGUE (after_prologue); +#endif return after_prologue == func_start; } else diff --git a/gdb/sparc-tdep.c b/gdb/sparc-tdep.c index a59986dd882..b718e4c8883 100644 --- a/gdb/sparc-tdep.c +++ b/gdb/sparc-tdep.c @@ -5,19 +5,19 @@ This file is part of GDB. -GDB is free software; you can redistribute it and/or modify +This program is free software; you can redistribute it and/or modify it under the terms of the GNU General Public License as published by -the Free Software Foundation; either version 1, or (at your option) -any later version. +the Free Software Foundation; either version 2 of the License, or +(at your option) any later version. -GDB is distributed in the hope that it will be useful, +This program is distributed in the hope that it will be useful, but WITHOUT ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License for more details. You should have received a copy of the GNU General Public License -along with GDB; see the file COPYING. If not, write to -the Free Software Foundation, 675 Mass Ave, Cambridge, MA 02139, USA. */ +along with this program; if not, write to the Free Software +Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA. */ #include #include "defs.h" @@ -127,9 +127,13 @@ sparc_frame_chain (thisframe) FRAME thisframe; { CORE_ADDR retval; - read_memory ((CORE_ADDR)&(((struct rwindow *)(thisframe->frame))->rw_in[6]), + int err; + err = target_read_memory + ((CORE_ADDR)&(((struct rwindow *)(thisframe->frame))->rw_in[6]), &retval, sizeof (CORE_ADDR)); + if (err) + return 0; return retval; } @@ -186,7 +190,7 @@ setup_arbitrary_frame (frame, stack) fatal ("internal: create_new_frame returned invalid frame id"); fid->bottom = stack; - + fid->pc = FRAME_SAVED_PC (fid); return fid; } @@ -286,11 +290,18 @@ do_restore_insn () restore_inferior_status (&inf_status); } -/* This routine should be more specific in it's actions; making sure +/* Given a pc value, skip it forward past the function prologue by + disassembling instructions that appear to be a prologue. + + If FRAMELESS_P is set, we are only testing to see if the function + is frameless. This allows a quicker answer. + + This routine should be more specific in its actions; making sure that it uses the same register in the initial prologue section. */ CORE_ADDR -skip_prologue (start_pc) +skip_prologue (start_pc, frameless_p) CORE_ADDR start_pc; + int frameless_p; { union { @@ -315,8 +326,6 @@ skip_prologue (start_pc) } x; int dest = -1; CORE_ADDR pc = start_pc; - /* Have we found a save instruction? */ - int found_save = 0; x.i = read_memory_integer (pc, 4); @@ -345,10 +354,16 @@ skip_prologue (start_pc) as there isn't any sign extension). */ if (x.add.op == 2 && (x.add.op3 ^ 32) == 28) { - found_save = 1; pc += 4; + if (frameless_p) /* If the save is all we care about, */ + return pc; /* return before doing more work */ x.i = read_memory_integer (pc, 4); } + else + { + /* Without a save instruction, it's not a prologue. */ + return start_pc; + } /* Now we need to recognize stores into the frame from the input registers. This recognizes all non alternate stores of input @@ -365,11 +380,7 @@ skip_prologue (start_pc) pc += 4; x.i = read_memory_integer (pc, 4); } - if (found_save) - return pc; - else - /* Without a save instruction, it's not a prologue. */ - return start_pc; + return pc; } /* Check instruction at ADDR to see if it is an annulled branch. diff --git a/gdb/tm-sparc.h b/gdb/tm-sparc.h index 3f3f2e4d8e7..7f0fb4e6fba 100644 --- a/gdb/tm-sparc.h +++ b/gdb/tm-sparc.h @@ -64,10 +64,15 @@ Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA. */ #define FUNCTION_START_OFFSET 0 /* Advance PC across any function entry prologue instructions - to reach some "real" code. */ + to reach some "real" code. SKIP_PROLOGUE_FRAMELESS_P advances + the PC past some of the prologue, but stops as soon as it + knows that the function has a frame. Its result is equal + to its input PC if the function is frameless, unequal otherwise. */ #define SKIP_PROLOGUE(pc) \ - { pc = skip_prologue (pc); } + { pc = skip_prologue (pc, 0); } +#define SKIP_PROLOGUE_FRAMELESS_P(pc) \ + { pc = skip_prologue (pc, 1); } extern CORE_ADDR skip_prologue (); /* Immediately after a function call, return the saved pc. @@ -570,7 +575,7 @@ extern void single_step (); /* We need two arguments (in general) to the "info frame" command. Note that the definition of this macro implies that there exists a - function "setup_arbitrary_frame" in mach-dep.c */ + function "setup_arbitrary_frame" in sparc-tdep.c */ #define FRAME_SPECIFICATION_DYADIC