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]

Re: find_saved_register(frame) -> next/prev frame?


Michael Snyder writes:
 > Andrew Cagney wrote:
 > > 
 > > > It seems that find_saved_register is called only by mips_get_saved_register()
 > > > and default_get_saved_register().
 > > >
 > > > There is another function, generic_get_saved_register() in
 > > > blockframe.c which is basically identical to
 > > > default_get_saved_register 
 > 
 > Not at all!  generic_get_saved_register is a special
 > purpose function, and if you look at the code you 
 > should be able to see that it is very different
 > from default_get_saved_register.
 > 

Can you please explain it to me. I am obviously missing something,
then.  The comments are not helping in explaining the difference, and
the name 'generic' doesn't indicate what the special purpose is.  How
does one choose to use one versus the other? It seems that all the
multiarched targets are using the generic version. And the
non-multiarched one are using the default version.

Thanks
Elena

 > > > and which doesn't call
 > > > find_saved_register(). Maybe the author realized that
 > > > find_saved_register() was wrong, but didn't want to touch the exisitng
 > > > code.
 > > 
 > > This is probably true (Hi mike :-).  
 > 
 > In fact it's not true.
 > 
 > > However, only a few targets are
 > > using that new version.  Most targets appear to still use
 > > default_get_saved_register().  Surely someone will have noticed that
 > > up/down didn't work right?
 > > 
 > > > Probably we should switch the macro GET_SAVED_REGISTER to be
 > > > generic_get_saved_register instead of default_get_saved_register and
 > > > see what happens.
 > > 
 > > If we can demonstrate that it is wrong then yes.
 > > 
 > > > Elena
 > > 
 > >         Andrew

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