From mboxrd@z Thu Jan 1 00:00:00 1970 From: Randolph Chung To: Andrew Cagney Cc: gdb-patches@sources.redhat.com Subject: Re: [patch/rfa] Some fixes for hppa fallback unwinder Date: Wed, 26 May 2004 04:02:00 -0000 Message-id: <20040526040215.GS7207@tausq.org> References: <20040522063442.GJ7207@tausq.org> <40B38C6B.2070603@gnu.org> <20040525183543.GI7207@tausq.org> <40B3B789.2030203@gnu.org> X-SW-Source: 2004-05/msg00727.html > >i guess that works too. should i move it there? > > Yes, feel free to commit that. thanks. committed. 2004-05-25 Randolph Chung * hppa-tdep.c (hppa_fallback_frame_cache): Handle stack adjustment, and multiple stack unwinds. (hppa_stub_unwind_sniffer): Handle unwind from pc == 0. Index: hppa-tdep.c =================================================================== RCS file: /cvs/src/src/gdb/hppa-tdep.c,v retrieving revision 1.164 diff -u -p -r1.164 hppa-tdep.c --- hppa-tdep.c 25 May 2004 03:55:23 -0000 1.164 +++ hppa-tdep.c 26 May 2004 03:56:38 -0000 @@ -1887,6 +1887,7 @@ static struct hppa_frame_cache * hppa_fallback_frame_cache (struct frame_info *next_frame, void **this_cache) { struct hppa_frame_cache *cache; + unsigned int frame_size; CORE_ADDR pc, start_pc, end_pc, cur_pc; cache = FRAME_OBSTACK_ZALLOC (struct hppa_frame_cache); @@ -1895,6 +1896,7 @@ hppa_fallback_frame_cache (struct frame_ pc = frame_func_unwind (next_frame); cur_pc = frame_pc_unwind (next_frame); + frame_size = 0; find_pc_partial_function (pc, NULL, &start_pc, &end_pc); @@ -1914,21 +1916,18 @@ hppa_fallback_frame_cache (struct frame_ insn = read_memory_unsigned_integer (pc, 4); + frame_size += prologue_inst_adjust_sp (insn); + /* There are limited ways to store the return pointer into the stack. */ if (insn == 0x6bc23fd9) /* stw rp,-0x14(sr0,sp) */ - { - cache->saved_regs[HPPA_RP_REGNUM].addr = -20; - break; - } + cache->saved_regs[HPPA_RP_REGNUM].addr = -20; else if (insn == 0x0fc212c1) /* std rp,-0x10(sr0,sp) */ - { - cache->saved_regs[HPPA_RP_REGNUM].addr = -16; - break; - } + cache->saved_regs[HPPA_RP_REGNUM].addr = -16; } - cache->base = frame_unwind_register_unsigned (next_frame, HPPA_SP_REGNUM); + cache->base = frame_unwind_register_unsigned (next_frame, HPPA_SP_REGNUM) - frame_size; + trad_frame_set_value (cache->saved_regs, HPPA_SP_REGNUM, cache->base); if (trad_frame_addr_p (cache->saved_regs, HPPA_RP_REGNUM)) { @@ -2080,7 +2079,8 @@ hppa_stub_unwind_sniffer (struct frame_i { CORE_ADDR pc = frame_pc_unwind (next_frame); - if (IN_SOLIB_CALL_TRAMPOLINE (pc, NULL) + if (pc == 0 + || IN_SOLIB_CALL_TRAMPOLINE (pc, NULL) || IN_SOLIB_RETURN_TRAMPOLINE (pc, NULL)) return &hppa_stub_frame_unwind; return NULL;