Richard Earnshaw <rearnsha@buzzard.freeserve.co.uk> writes:

> > > Then the current code setcpsr.S code is screwed because set_cpsr can
> > > clobber "r8","r9","r10","r11","r12","r13","r14".
> > 
> > This is only true if it's ever used to change mode, which I'm fairly sure
> > it isn't.
> 
> And it certainly shouldn't ever be used to change out of FIQ mode
> (or we've got the OS design wrong).

Would you prefer it to clobber fewer registers and document that you
may not switch from FIQ mode? I didn't see any harm in clobbering more
registers . . .

> PS.  You are probably missing a clobber of CC as well.

Whoops. Yes. Fixed now.

-- 

	http://ape.n3.net