Revision: 8051
Author: [email protected]
Date: Tue May 24 23:41:39 2011
Log: MIPS: Fixed two bugs related to double function calls.
These originated from 4dfb7f2e.
This fixes cctest test-api/ConversionNumber in soft-float mode.
BUG=
TEST=
Review URL: http://codereview.chromium.org//7068002
http://code.google.com/p/v8/source/detail?r=8051
Modified:
/branches/bleeding_edge/src/mips/code-stubs-mips.cc
/branches/bleeding_edge/src/mips/macro-assembler-mips.cc
=======================================
--- /branches/bleeding_edge/src/mips/code-stubs-mips.cc Tue May 24 23:04:01
2011
+++ /branches/bleeding_edge/src/mips/code-stubs-mips.cc Tue May 24 23:41:39
2011
@@ -3439,9 +3439,9 @@
&call_runtime);
__ push(ra);
__ PrepareCallCFunction(3, scratch);
- __ SetCallCDoubleArguments(double_base, double_exponent);
+ __ SetCallCDoubleArguments(double_base, exponent);
__ CallCFunction(
- ExternalReference::power_double_int_function(masm->isolate()), 4);
+ ExternalReference::power_double_int_function(masm->isolate()), 3);
__ pop(ra);
__ GetCFunctionDoubleResult(double_result);
__ sdc1(double_result,
=======================================
--- /branches/bleeding_edge/src/mips/macro-assembler-mips.cc Tue May 24
23:04:01 2011
+++ /branches/bleeding_edge/src/mips/macro-assembler-mips.cc Tue May 24
23:41:39 2011
@@ -2651,9 +2651,9 @@
void MacroAssembler::GetCFunctionDoubleResult(const DoubleRegister dst) {
CpuFeatures::Scope scope(FPU);
if (IsMipsSoftFloatABI) {
- Move(v0, v1, dst);
+ Move(dst, v0, v1);
} else {
- Move(f0, dst); // Reg f0 is o32 ABI FP return value.
+ Move(dst, f0); // Reg f0 is o32 ABI FP return value.
}
}
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
