STUB_MOVE in elfxx-mips.c

[Thiemo Seufer]

Eric Christopher wrote:
> 
> > 
> > I don't think that's actually a problem. My proposed patch only changes
> > the stub handling, which is about computing addresses. Anything else
> > than ABI_64_P will use 32bit values, and the ABI_64_P case seems to be
> > broken for quite some time now.
> > 
> > I don't want to change the implementation of 'move' without good reason.
> 
> This is a good reason. The current definition has side effects and
> requires things that or does not.

AFAICS the _current_ definition in CVS handles everything fine,
without unwanted side effects.

> I _thought_ I'd changed it at some
> point, but apparently not. I've been running with or as my move for
> quite some time.

It was changed to addu/daddu:
http://sources.redhat.com/ml/binutils/2001-08/msg00117.html

Is there any reason to change it once again, two years later?


Thiemo