>> Looks good to me.  However, patches like this one break pure
>> multi-arch targets that are converted to use the non-deprecated
>> variants of these functions if they don't fill in the deprecated
>> function in their gdbarch too.

> My idea for fixing this is illustrated by the following patch, but
> perhaps there is a more elegant way to do this?

I've so far not come up with anything better.  The attached gets around 
the problem by dropping the sanity check -- it is easier to apply.  The 
lack of a function will be detected (internal-error) but when it is 
first called.

thoughts?
Andrew