Fix load/store of external float arrays on ARM

Fix load/store of external float arrays on ARM


BUG=1323
TEST=mjsunit/regress/regress-1323.js, run with simulator=arm

Committed: http://code.google.com/p/v8/source/detail?r=7590

[Jakob Kummerow, 2011-04-12 14:41:28]

PTAL

[danno, 2011-04-12 15:10:50]

http://codereview.chromium.org/6822054/diff/1/src/arm/lithium-codegen-arm.cc
File src/arm/lithium-codegen-arm.cc (right):

http://codereview.chromium.org/6822054/diff/1/src/arm/lithium-codegen-arm.cc#...
src/arm/lithium-codegen-arm.cc:3275: __ vstr(value.low(), scratch0(), 0);
You need a corresponding UseTempRegister since you change value, or use a
scratch value.

[Jakob Kummerow, 2011-04-12 15:13:39]

http://codereview.chromium.org/6822054/diff/1/src/arm/lithium-codegen-arm.cc
File src/arm/lithium-codegen-arm.cc (right):

http://codereview.chromium.org/6822054/diff/1/src/arm/lithium-codegen-arm.cc#...
src/arm/lithium-codegen-arm.cc:3275: __ vstr(value.low(), scratch0(), 0);
On 2011/04/12 15:10:50, danno wrote:
> You need a corresponding UseTempRegister since you change value, or use a
> scratch value.

Done.

[danno, 2011-04-12 15:20:01]

LGTM