[PATCH,MIPS 3/3] Setting of status register based on .MIPS.abiflags

[Corinna Vinschen]

On Nov 27 23:46, Matthew Fortune wrote:
> Steve Ellcey <Steve.Ellcey@imgtec.com> writes:
> > > On Tue, 2014-11-18 at 12:26 +0000, Matthew Fortune wrote:
> > >
> > > > +	/* Read MIPS_abiflags structure and set status/config registers
> > > > +	   accordingly.  */
> > > > +	.weak	__MIPS_abiflags_start
> > > > +	.weak	__MIPS_abiflags_end
> > > > +	LA	(t0,__MIPS_abiflags_start)
> > > > +	LA	(t1,__MIPS_abiflags_end)
> > > > +	addiu	t1,t1,-24
> > > > +	move	v0,zero			/* Mask for C0_SR.  */
> > > > +	bne	t0,t1,1f
> > >
> > > I would add a bit more to the comment here to explicitly say that we
> > > are checking to see if the abiflags structure is defined and jumping
> > to '1f'
> > > if it is not.
> > >
> > > Other then that the patch looked OK to me.
> > 
> > OK, I agree the test is a bit odd. I'll send an updated patch in the
> > next day or so.
> 
> Updated patch attached with a comment at the appropriate place. Let me
> know if you think anything else needs work.

Patch applied.


Thanks,
Corinna

-- 
Corinna Vinschen
Cygwin Maintainer
Red Hat
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://sourceware.org/pipermail/newlib/attachments/20141128/80ff5de4/attachment.sig>