Add Wasm SIMD opcodes.

Add Wasm SIMD opcodes.
  - Add Simd128 type to Wasm AST types
  - Seperate out generic SIMD Machine ops
  - Opcodes added as per Wasm SIMD opcode table prototype. All opcodes
  except Bool8x16, Bool16x8 and S128 Load/Store opcodes added.

Committed: https://crrev.com/0f777211d72a10d976d55a490f4f2b6e3e4bb695
Cr-Commit-Position: refs/heads/master@{#36506}

[gdeepti, 2016-05-24 16:59:05]

Description was changed from

==========
Add Wasm SIMD opcodes.
  - Add Simd128 type to Wasm AST types
  - Seperate out generic SIMD Machine ops
  - Opcodes added as per Wasm SIMD opcode table prototype. All opcodes
  except Bool8x16, Bool16x8 and S128 Load/Store opcodes added.
==========

to

==========
Add Wasm SIMD opcodes.
  - Add Simd128 type to Wasm AST types
  - Seperate out generic SIMD Machine ops
  - Opcodes added as per Wasm SIMD opcode table prototype. All opcodes
  except Bool8x16, Bool16x8 and S128 Load/Store opcodes added.
==========

[gdeepti, 2016-05-24 16:59:05]

gdeepti@chromium.org changed reviewers:
+ bbudge@chromium.org, bradnelson@chromium.org, titzer@chromium.org

[bradnelson, 2016-05-24 19:47:13]

lgtm, but need ben for owners

[titzer, 2016-05-25 07:38:39]

lgtm other than swizzle signatures

https://codereview.chromium.org/2009553002/diff/1/src/wasm/wasm-opcodes.h
File src/wasm/wasm-opcodes.h (right):

https://codereview.chromium.org/2009553002/diff/1/src/wasm/wasm-opcodes.h#new...
src/wasm/wasm-opcodes.h:362: V(I16x8Swizzle, 0xe54d, s_siiii)     \
Are the inputs to Swizzle and Shuffle supposed to be constants? If so, then they
should be encoded as immediates.

[gdeepti, 2016-05-25 09:17:51]

https://codereview.chromium.org/2009553002/diff/1/src/wasm/wasm-opcodes.h
File src/wasm/wasm-opcodes.h (right):

https://codereview.chromium.org/2009553002/diff/1/src/wasm/wasm-opcodes.h#new...
src/wasm/wasm-opcodes.h:362: V(I16x8Swizzle, 0xe54d, s_siiii)     \
On 2016/05/25 07:38:39, titzer wrote:
> Are the inputs to Swizzle and Shuffle supposed to be constants? If so, then
they
> should be encoded as immediates.

Fixed shuffle, swizzle signatures and added missing opcodes for I8x16Splat,
I16x8Splat.

[gdeepti, 2016-05-25 09:18:55]

The CQ bit was checked by gdeepti@chromium.org

[gdeepti, 2016-05-25 09:18:55]

The patchset sent to the CQ was uploaded after l-g-t-m from titzer@chromium.org,
bradnelson@chromium.org
Link to the patchset: https://codereview.chromium.org/2009553002/#ps20001
(title: "Fix shuffle/swizzle signatures")

[commit-bot: I haz the power, 2016-05-25 09:18:57]

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

[commit-bot: I haz the power, 2016-05-25 09:54:15]

Description was changed from

==========
Add Wasm SIMD opcodes.
  - Add Simd128 type to Wasm AST types
  - Seperate out generic SIMD Machine ops
  - Opcodes added as per Wasm SIMD opcode table prototype. All opcodes
  except Bool8x16, Bool16x8 and S128 Load/Store opcodes added.
==========

to

==========
Add Wasm SIMD opcodes.
  - Add Simd128 type to Wasm AST types
  - Seperate out generic SIMD Machine ops
  - Opcodes added as per Wasm SIMD opcode table prototype. All opcodes
  except Bool8x16, Bool16x8 and S128 Load/Store opcodes added.
==========

[commit-bot: I haz the power, 2016-05-25 09:54:16]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-05-25 09:56:55]

Description was changed from

==========
Add Wasm SIMD opcodes.
  - Add Simd128 type to Wasm AST types
  - Seperate out generic SIMD Machine ops
  - Opcodes added as per Wasm SIMD opcode table prototype. All opcodes
  except Bool8x16, Bool16x8 and S128 Load/Store opcodes added.
==========

to

==========
Add Wasm SIMD opcodes.
  - Add Simd128 type to Wasm AST types
  - Seperate out generic SIMD Machine ops
  - Opcodes added as per Wasm SIMD opcode table prototype. All opcodes
  except Bool8x16, Bool16x8 and S128 Load/Store opcodes added.

Committed: https://crrev.com/0f777211d72a10d976d55a490f4f2b6e3e4bb695
Cr-Commit-Position: refs/heads/master@{#36506}
==========

[commit-bot: I haz the power, 2016-05-25 09:56:56]

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

[aseemgarg, 2016-05-27 23:59:37]

aseemgarg@chromium.org changed reviewers:
+ aseemgarg@chromium.org

[aseemgarg, 2016-05-27 23:59:37]

https://codereview.chromium.org/2009553002/diff/20001/src/wasm/wasm-opcodes.h
File src/wasm/wasm-opcodes.h (right):

https://codereview.chromium.org/2009553002/diff/20001/src/wasm/wasm-opcodes.h...
src/wasm/wasm-opcodes.h:286: V(F32x4ExtractLane, 0xe501, f_si)    \
probably better to take this and similar opcodes out into separate Macro and
implement them without the signature (they will become separate cases in
DecodeFunctionBody). The lane id should be an an operand instead of a parameter.

https://codereview.chromium.org/2009553002/diff/20001/src/wasm/wasm-opcodes.h...
src/wasm/wasm-opcodes.h:329: V(I32x4Select, 0xe52c, s_sss)        \
These won't work. There is no implementation for opcodes with 3 parameters.