Ulrich Weigand schrieb: > Markus Deuling wrote: > >> * mn10300-tdep.c (set_reg_offsets,mn10300_analyze_prologue): Replace >> AM33 by its expression. Use get_frame_arch to get at the current >> architecture by frame_info. >> * mn10300-tdep.h (AM33_MODE): Remove. > > Sorry, but this is not OK. Unfortunately, the mn10300_analyze_prologue > routine is called both from within the unwinder (where we always have > a frame), *and* from mn10300_skip_prologue, where frame information is > not available. Thus "fi" may actually be NULL. Ok, I see. Your're right. I attached a patch which just replaces AM33_MODe macro to get rid of current_gdbarch in mn10300-tdep.h file. Later on I'll come up with a patch to get rid of the newly introduced current_gdbarch's in mn10300-tdep.c. Tested by building with --target=mn10300. Ok to commit? ChangeLog: * mn10300-tdep.c (set_reg_offsets,mn10300_analyze_prologue): Replace AM33 by its expression. * mn10300-tdep.h (AM33_MODE): Remove. -- Markus Deuling GNU Toolchain for Linux on Cell BE deuling@de.ibm.com