MIPS: Remove gp and sp from callee-saved register list.

MIPS: Remove gp and sp from callee-saved register list.


BUG=
TEST=
Committed: http://code.google.com/p/v8/source/detail?r=9020

[fschneider, 2011-08-25 10:50:51]

http://codereview.chromium.org/7740018/diff/1/src/mips/code-stubs-mips.cc
File src/mips/code-stubs-mips.cc (right):

http://codereview.chromium.org/7740018/diff/1/src/mips/code-stubs-mips.cc#new...
src/mips/code-stubs-mips.cc:3833: __ MultiPop((kCalleeSaved | ra.bit()) &
~sp.bit());
I guess you have to adjust the corresponding Multipop as well, right?

__ MultiPop((kCalleeSaved | ra.bit());

[Paul Lind, 2011-08-25 15:47:50]

http://codereview.chromium.org/7740018/diff/1/src/mips/code-stubs-mips.cc
File src/mips/code-stubs-mips.cc (right):

http://codereview.chromium.org/7740018/diff/1/src/mips/code-stubs-mips.cc#new...
src/mips/code-stubs-mips.cc:3833: __ MultiPop((kCalleeSaved | ra.bit()) &
~sp.bit());
On 2011/08/25 10:50:51, fschneider wrote:
> I guess you have to adjust the corresponding Multipop as well, right?
> 
> __ MultiPop((kCalleeSaved | ra.bit());

Yes, of course you're right. Leaving it didn't cause tests to fail (since we are
removing sp.bit(), which was already removed from kCalleeSaved), which is how we
missed it. But it makes the code look broken. Thanks for catching this. Fixed.

[fschneider, 2011-08-26 07:11:09]

LGTM.

I'll land it.