This is the mail archive of the gdb-patches@sources.redhat.com mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [RFA] update alpha return value hooks to regcache


Richard Henderson <rth@twiddle.net> writes:

> Ok?
> 
> r~
> 
> 	* alpha-tdep.c (alpha_extract_return_value): Convert to regcache.
> 	(alpha_extract_struct_value_address): Likewise.
> 	(alpha_store_return_value): Likewise.
> 	(alpha_store_struct_return): Remove.
> 	(alpha_gdbarch_init): Update hook registration to match.
>

Ah, this one's got a flaw.  Using abort() isn't allowed in GDB.
Instead use internal_error() which gives the user the option of
continuing at his/her own risk.  You could also use gdb_assert()
(which calls internal_error()).

With that change, this is approved.

Mark


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]