Re: [PATCH]: alpha_frame_saved_pc()

[Andrew Cagney <ac131313 at cygnus dot com>]

Michael Snyder wrote:
> 
> Hey, I don't know who the alpha maintainer is!
> 
> Does this change look good?

There isn't an active alpha maintainer -> me -> just use your judgment. 
Do you have access to some sort of alpha machine?

	Andrew


> --- alpha-tdep.c (alpha_frame_saved_pc):
> ***************
> *** 381,386 ****
> --- 509,515 ----
>        struct frame_info *frame;
>   {
>     alpha_extra_func_info_t proc_desc = frame->proc_desc;
> +   CORE_ADDR saved_pc;
>     /* We have to get the saved pc from the sigcontext
>        if it is a signal handler frame.  */
>     int pcreg = frame->signal_handler_caller ? PC_REGNUM :
> frame->pc_reg;
> ***************
> *** 388,394 ****
>     if (proc_desc && PROC_DESC_IS_DUMMY (proc_desc))
>       return read_memory_integer (frame->frame - 8, 8);
> 
> !   return read_next_frame_reg (frame, pcreg);
>   }
> 
>   CORE_ADDR
> --- 517,526 ----
>     if (proc_desc && PROC_DESC_IS_DUMMY (proc_desc))
>       return read_memory_integer (frame->frame - 8, 8);
> 
> !   saved_pc = read_next_frame_reg (frame, pcreg);
> !   if (saved_pc == 0)
> !     saved_pc = read_next_frame_reg (frame, RA_REGNUM);
> !   return saved_pc;
>   }
> 
>   CORE_ADDR