This is the mail archive of the binutils@sources.redhat.com mailing list for the binutils project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: STUB_MOVE in elfxx-mips.c


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


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