Ulrich Weigand schrieb:
> Markus Deuling wrote:
> 
>> @@ -604,7 +606,7 @@ mn10300_analyze_prologue (struct frame_i
> 
>> -  if (AM33_MODE == 2)
>> +  if (AM33_MODE (get_frame_arch (fi))== 2)
> 
> This is incorrect, as mn10300_analyze_prologue may be called 
> with a NULL fi value (from mn10300_skip_prologue).  I guess
> the right way would be to allow the caller to pass an explicit
> gdbarch ...
> 

Hi Uli,

thank you for review. I attached a new version of the patch. Tested by building mn10300 target
(--enable-targets=all). Ok to commit ?


ChangeLog:

	* mn10300-tdep.h (AM33_MODE): Add gdbarch as parameter.
	* mn10300-tdep.c (set_reg_offsets, mn10300_analyze_prologue): Use
	get_frame_arch to get at the current_architecture. Update AM33_MODE
	call.
	(mn10300_analyze_prologue): Add gdbarch as parameter. Update caller.
	(mn10300_frame_unwind_cache): Use get_frame_arch to get at the current
	architecture.
	(set_reg_offsets, mn10300_analyze_prologue): Fix indentation.



-- 
  Markus Deuling
  GNU Toolchain for Linux on Cell BE
  deuling@de.ibm.com