MIPS[64]: Support for some SIMD operations

MIPS[64]: Support for some SIMD operations

Adds support for I32x4Splat, I32x4ExtractLane, I32x4ReplaceLane,
I32x4Add, I32x4Sub, S128Zero operations for mips32 and mips64
architectures.

BUG=

Note: Depends on patch: https://codereview.chromium.org/2740123004/
Review-Url: https://codereview.chromium.org/2753903004
Cr-Commit-Position: refs/heads/master@{#44326}
Committed: https://chromium.googlesource.com/v8/v8/+/12faf0f87f0d180abdfe294f6646bbf2044789e8

[dusan.simicic, 2017-03-17 14:09:17]

dusan.simicic@imgtec.com changed reviewers:
+ ivica.bogosavljevic@imgtec.com, miran.karic@imgtec.com,
v8-mips-ports@googlegroups.com

[dusan.simicic, 2017-03-17 14:09:17]

PTAL

[ivica.bogosavljevic, 2017-03-21 10:45:27]

lgtm

[dusan.simicic, 2017-03-27 13:30:17]

dusan.simicic@imgtec.com changed reviewers:
+ bbudge@chromium.org, titzer@chromium.org

[dusan.simicic, 2017-03-27 13:30:18]

+titzer, +bbudge

[bbudge, 2017-03-27 17:12:03]

lgtm

https://codereview.chromium.org/2753903004/diff/60001/test/cctest/wasm/test-r...
File test/cctest/wasm/test-run-wasm-simd.cc (right):

https://codereview.chromium.org/2753903004/diff/60001/test/cctest/wasm/test-r...
test/cctest/wasm/test-run-wasm-simd.cc:39: #endif  // !V8_TARGET_ARCH_ARM &&
!V8_TARGET_ARCH_X64 && !V8_TARGET_ARCH_IA32
nit: add mips defines to comment, or just remove the comment.

[dusan.simicic, 2017-03-28 08:59:01]

https://codereview.chromium.org/2753903004/diff/60001/test/cctest/wasm/test-r...
File test/cctest/wasm/test-run-wasm-simd.cc (right):

https://codereview.chromium.org/2753903004/diff/60001/test/cctest/wasm/test-r...
test/cctest/wasm/test-run-wasm-simd.cc:39: #endif  // !V8_TARGET_ARCH_ARM &&
!V8_TARGET_ARCH_X64 && !V8_TARGET_ARCH_IA32
On 2017/03/27 17:12:03, bbudge wrote:
> nit: add mips defines to comment, or just remove the comment.

#if and #endif are close enough, I'll remove the comment.

[dusan.simicic, 2017-03-29 10:13:58]

The CQ bit was checked by dusan.simicic@imgtec.com to run a CQ dry run

[commit-bot: I haz the power, 2017-03-29 10:14:04]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-03-29 11:29:23]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-03-29 11:29:24]

Dry run: This issue passed the CQ dry run.

[dusan.simicic, 2017-03-29 12:21:38]

The CQ bit was checked by dusan.simicic@imgtec.com

[dusan.simicic, 2017-03-29 12:21:39]

The patchset sent to the CQ was uploaded after l-g-t-m from
ivica.bogosavljevic@imgtec.com, bbudge@chromium.org
Link to the patchset: https://codereview.chromium.org/2753903004/#ps80001
(title: "removed comment in test-run-wasm-simd.cc")

[commit-bot: I haz the power, 2017-03-29 12:21:47]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-03-29 12:24:37]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-03-29 12:24:38]

Try jobs failed on following builders:
  v8_presubmit on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_presubmit/builds/37961)

[dusan.simicic, 2017-03-29 12:29:44]

dusan.simicic@imgtec.com changed reviewers:
+ mtrofin@chromium.org

[dusan.simicic, 2017-03-29 12:29:45]

+ mtrofin for src/compiler/instruction-selector.cc

[dusan.simicic, 2017-03-31 12:51:15]

Description was changed from

==========
MIPS[64]: Support for some SIMD operations

Adds support for Int32x4Splat, Int32x4ExtractLane, Int32x4ReplaceLane,
Int32x4Add, Int32x4Sub, Simd128Zero operations for mips32 and mips64
architectures.

BUG=

Note: Depends on patch: https://codereview.chromium.org/2740123004/
==========

to

==========
MIPS[64]: Support for some SIMD operations

Adds support for I32x4Splat, I32x4ExtractLane, I32x4ReplaceLane,
I32x4Add, I32x4Sub, S128Zero operations for mips32 and mips64
architectures.

BUG=

Note: Depends on patch: https://codereview.chromium.org/2740123004/
==========

[dusan.simicic, 2017-03-31 12:54:25]

PS6: rebased and changed opcode names according to
https://codereview.chromium.org/2776753004/

PTAL again

[Mircea Trofin, 2017-03-31 15:14:33]

On 2017/03/31 12:54:25, dusan.simicic wrote:
> PS6: rebased and changed opcode names according to
> https://codereview.chromium.org/2776753004/
> 
> PTAL again

rubberstamp lgtm for the instruction-selector part.

[bbudge, 2017-03-31 16:20:32]

Still LGTM.

[dusan.simicic, 2017-04-03 05:53:34]

The CQ bit was checked by dusan.simicic@imgtec.com to run a CQ dry run

[commit-bot: I haz the power, 2017-04-03 05:53:39]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-04-03 05:54:49]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-04-03 05:54:50]

Dry run: Try jobs failed on following builders:
  v8_linux_nodcheck_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_nodcheck_rel_ng/bu...)
  v8_mac_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_mac_rel_ng/builds/20220)
  v8_presubmit on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_presubmit/builds/38239)

[dusan.simicic, 2017-04-03 07:04:12]

The CQ bit was checked by dusan.simicic@imgtec.com to run a CQ dry run

[commit-bot: I haz the power, 2017-04-03 07:04:22]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-04-03 08:08:12]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-04-03 08:08:15]

Dry run: This issue passed the CQ dry run.

[dusan.simicic, 2017-04-03 08:20:38]

The CQ bit was checked by dusan.simicic@imgtec.com

[dusan.simicic, 2017-04-03 08:20:38]

The patchset sent to the CQ was uploaded after l-g-t-m from
ivica.bogosavljevic@imgtec.com, bbudge@chromium.org, mtrofin@chromium.org
Link to the patchset: https://codereview.chromium.org/2753903004/#ps120001
(title: "rebased")

[commit-bot: I haz the power, 2017-04-03 08:20:42]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-04-03 08:22:20]

CQ is committing da patch.

Bot data: {"patchset_id": 120001, "attempt_start_ts": 1491207638524310,
"parent_rev": "3258b2690e1be50e2ab47722585ecfc29754a657", "commit_rev":
"12faf0f87f0d180abdfe294f6646bbf2044789e8"}

[commit-bot: I haz the power, 2017-04-03 08:22:24]

Description was changed from

==========
MIPS[64]: Support for some SIMD operations

Adds support for I32x4Splat, I32x4ExtractLane, I32x4ReplaceLane,
I32x4Add, I32x4Sub, S128Zero operations for mips32 and mips64
architectures.

BUG=

Note: Depends on patch: https://codereview.chromium.org/2740123004/
==========

to

==========
MIPS[64]: Support for some SIMD operations

Adds support for I32x4Splat, I32x4ExtractLane, I32x4ReplaceLane,
I32x4Add, I32x4Sub, S128Zero operations for mips32 and mips64
architectures.

BUG=

Note: Depends on patch: https://codereview.chromium.org/2740123004/
Review-Url: https://codereview.chromium.org/2753903004
Cr-Commit-Position: refs/heads/master@{#44326}
Committed:
https://chromium.googlesource.com/v8/v8/+/12faf0f87f0d180abdfe294f6646bbf2044...
==========

[commit-bot: I haz the power, 2017-04-03 08:22:26]

Committed patchset #7 (id:120001) as
https://chromium.googlesource.com/v8/v8/+/12faf0f87f0d180abdfe294f6646bbf2044...