[PATCH,MIPS 3/3] Setting of status register based on .MIPS.abiflags
Matthew Fortune
Matthew.Fortune@imgtec.com
Thu Nov 27 23:46:00 GMT 2014
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.
Thanks,
Matthew
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0003-Setting-of-status-register-based-on-.MIPS.abiflags-s.patch
Type: application/octet-stream
Size: 10783 bytes
Desc: 0003-Setting-of-status-register-based-on-.MIPS.abiflags-s.patch
URL: <http://sourceware.org/pipermail/newlib/attachments/20141127/8d5ef1e9/attachment.obj>
More information about the Newlib
mailing list