> This is a following on the following change: > > http://sources.redhat.com/ml/gdb-patches/2004-03/msg00000.html > > GDB fails to build on AiX because we're trying to dereference an > opaque structure (struct frame_info). > > I suggest the following patch to fix the build failure: > > * rs6000-tdep.c (rs6000_init_frame_pc_first): Fix compilation failure. > > OK to apply? (it may appear obvious to the usual maintainers, but the > frame stuff still makes me nervous sometimes, especially the easy > confusion with next, prev, inner most, etc...) Bah! The check that this built must have got lost between my machines. I've checked in the attached. Andrew > Index: rs6000-tdep.c > =================================================================== > RCS file: /cvs/src/src/gdb/rs6000-tdep.c,v > retrieving revision 1.182 > diff -u -p -r1.182 rs6000-tdep.c > --- rs6000-tdep.c 1 Mar 2004 00:50:56 -0000 1.182 > +++ rs6000-tdep.c 1 Mar 2004 20:02:05 -0000 > @@ -213,8 +213,10 @@ rs6000_frame_init_saved_regs (struct fra > static CORE_ADDR > rs6000_init_frame_pc_first (int fromleaf, struct frame_info *prev) > { > - return (fromleaf ? DEPRECATED_SAVED_PC_AFTER_CALL (prev->next) > - : prev->next ? DEPRECATED_FRAME_SAVED_PC (prev->next) : read_pc ()); > + struct frame_info* next = get_next_frame (prev); > + > + return (fromleaf ? DEPRECATED_SAVED_PC_AFTER_CALL (next) > + : next ? DEPRECATED_FRAME_SAVED_PC (next) : read_pc ()); > } > > static CORE_ADDR