[simd.js] Single SIMD128_VALUE_TYPE for all Simd128Values.

[simd.js] Single SIMD128_VALUE_TYPE for all Simd128Values.

There's no need to have one InstanceType per SIMD primitive type (this
will not scale long-term).  Also reduce the amount of code duplication
and make it more robust wrt adding new SIMD types.

R=yangguo@chromium.org

Committed: https://crrev.com/f4c079d450a5990639b295d40a3d1663d70412d6
Cr-Commit-Position: refs/heads/master@{#30107}

[Benedikt Meurer, 2015-08-11 08:52:42]

Hey Yang,

This reduces the number of instance types for SIMD primitives to 1 again
(otherwise we will run out of instance types at some point), and removes a lot
of code duplication.
Please take a look.

Thanks,
Benedikt

Bill: As discussed earlier today; FYI.
Andreas: FYI

[Yang, 2015-08-11 09:08:35]

On 2015/08/11 08:52:42, Benedikt Meurer wrote:
> Hey Yang,
> 
> This reduces the number of instance types for SIMD primitives to 1 again
> (otherwise we will run out of instance types at some point), and removes a lot
> of code duplication.
> Please take a look.
> 
> Thanks,
> Benedikt
> 
> Bill: As discussed earlier today; FYI.
> Andreas: FYI

LGTM with suggestion.

[Yang, 2015-08-11 09:08:46]

https://codereview.chromium.org/1273353003/diff/1/src/code-stubs-hydrogen.cc
File src/code-stubs-hydrogen.cc (right):

https://codereview.chromium.org/1273353003/diff/1/src/code-stubs-hydrogen.cc#...
src/code-stubs-hydrogen.cc:402: is_float32x4.If<HCompareObjectEqAndBranch>(
Can we do an instance type check first to skip forward to the object case?

We could then bail out to the runtime implementation for the simd case (if we
don't care about performance that much).

[Benedikt Meurer, 2015-08-11 09:11:46]

https://codereview.chromium.org/1273353003/diff/1/src/code-stubs-hydrogen.cc
File src/code-stubs-hydrogen.cc (right):

https://codereview.chromium.org/1273353003/diff/1/src/code-stubs-hydrogen.cc#...
src/code-stubs-hydrogen.cc:402: is_float32x4.If<HCompareObjectEqAndBranch>(
As discussed offline: There's no runtime fallback (yet). I will cleanup this
CodeStub in followup CLs.

[Benedikt Meurer, 2015-08-11 09:11:55]

The CQ bit was checked by bmeurer@chromium.org

[commit-bot: I haz the power, 2015-08-11 09:12:02]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1273353003/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1273353003/1

[Benedikt Meurer, 2015-08-11 09:19:01]

The CQ bit was checked by bmeurer@chromium.org

[Benedikt Meurer, 2015-08-11 09:19:01]

The patchset sent to the CQ was uploaded after l-g-t-m from yangguo@chromium.org
Link to the patchset: https://codereview.chromium.org/1273353003/#ps20001
(title: "Fix slow check failures. REBASE.")

[commit-bot: I haz the power, 2015-08-11 09:19:12]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1273353003/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1273353003/20001

[commit-bot: I haz the power, 2015-08-11 09:45:31]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-08-11 09:45:52]

Patchset 2 (id:??) landed as
https://crrev.com/f4c079d450a5990639b295d40a3d1663d70412d6
Cr-Commit-Position: refs/heads/master@{#30107}