A64: Add support for SMI representation to MulConstI. Also add a test.

A64: Add support for SMI representation to MulConstI. Also add a test.

R=jacob.bramley@arm.com

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

[rmcilroy, 2014-02-03 15:37:40]

SMI versions of MulConst won't be hit until we've re-based up to r16361, and
until crrev.com/151923004/ (or something like it) lands, but I've tested this
with both these changes patched in.

PTAL.

[jbramley, 2014-02-03 15:46:32]

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

https://codereview.chromium.org/152133003/diff/1/src/a64/lithium-codegen-a64....
src/a64/lithium-codegen-a64.cc:4156: void LCodeGen::DoMulConstI(LMulConstI*
instr) {
Since it now handles smis, could you rename it to LMulConstIS please?

[rmcilroy, 2014-02-03 15:52:31]

On 2014/02/03 15:46:32, jbramley wrote:
>
https://codereview.chromium.org/152133003/diff/1/src/a64/lithium-codegen-a64.cc
> File src/a64/lithium-codegen-a64.cc (right):
> 
>
https://codereview.chromium.org/152133003/diff/1/src/a64/lithium-codegen-a64....
> src/a64/lithium-codegen-a64.cc:4156: void LCodeGen::DoMulConstI(LMulConstI*
> instr) {
> Since it now handles smis, could you rename it to LMulConstIS please?

Yes, done.  I was wondering whether to add the S or remove the I :).

[jbramley, 2014-02-03 15:54:22]

lgtm

Good stuff.

[rmcilroy, 2014-02-03 16:49:19]

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