Ulrich Weigand schrieb: > Markus Deuling wrote: > >> * 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. > > This is OK, but please fix this small whitespace issue: > >> + if (AM33_MODE (gdbarch)== 2) > > Space between ')' and '=='. > >> + if (AM33_MODE (gdbarch)== 2) > > Likewise. > Hi Uli, thanks for review. I must have overseen that :-) I now committed this fixed patch attached. Regards, Markus -- Markus Deuling GNU Toolchain for Linux on Cell BE deuling@de.ibm.com