A64: Move integer exponent into correct register before calling MathPowStub.

A64: Move integer exponent into correct register before calling MathPowStub.

Follow-up to r18911.

TEST=mozilla/ecma/Expressions/11.7.1
R=baptiste.afsa@arm.com, jochen@chromium.org

Committed: https://code.google.com/p/v8/source/detail?r=18924

[ulan, 2014-01-29 15:13:41]

PTAL

https://codereview.chromium.org/136093005/diff/1/src/a64/lithium-codegen-a64.cc
File src/a64/lithium-codegen-a64.cc (right):

https://codereview.chromium.org/136093005/diff/1/src/a64/lithium-codegen-a64....
src/a64/lithium-codegen-a64.cc:3788: __ Mov(x12, x11);
MathPowStub expects integer exponent in x12

[jochen (gone - plz use gerrit), 2014-01-29 15:19:36]

lgtm

[baptiste.afsa1, 2014-01-29 15:23:48]

https://codereview.chromium.org/136093005/diff/1/src/a64/lithium-codegen-a64.cc
File src/a64/lithium-codegen-a64.cc (right):

https://codereview.chromium.org/136093005/diff/1/src/a64/lithium-codegen-a64....
src/a64/lithium-codegen-a64.cc:3788: __ Mov(x12, x11);
Isn't it possible to modify register constraints in lithium-a64.cc to avoid
doing this move explicitly?

[ulan, 2014-01-29 15:33:14]

Thanks! Uploaded new patch set, PTAL.

https://codereview.chromium.org/136093005/diff/1/src/a64/lithium-codegen-a64.cc
File src/a64/lithium-codegen-a64.cc (right):

https://codereview.chromium.org/136093005/diff/1/src/a64/lithium-codegen-a64....
src/a64/lithium-codegen-a64.cc:3788: __ Mov(x12, x11);
Agreed. Done.

On 2014/01/29 15:23:48, baptiste.afsa1 wrote:
> Isn't it possible to modify register constraints in lithium-a64.cc to avoid
> doing this move explicitly?

[baptiste.afsa1, 2014-01-29 15:37:19]

lgtm

[ulan, 2014-01-29 15:50:42]

Committed patchset #2 manually as r18924 (presubmit successful).