MIPS[64]: Fix deoptimizer generate table for large deoptimization tables

MIPS[64]: Fix deoptimizer generate table for large deoptimization tables

Fix 233b6926e3ae29a4567b4c55bc1c1e96c07c1f92

With the introduction of upfront deopt table generation, case where
deoptimization tables have more than 8000 entries become much more common
and bugs started to appear.

This CL fixes the issue, but it is far from perfect. We had to
increase number of instruction per table entry from 2 to 3 in
order to accommodate larger tables.

TEST=mjsunit/array-sort,mjsunit/json,mjsunit/md5
BUG=

Review-Url: https://codereview.chromium.org/2901393003
Cr-Commit-Position: refs/heads/master@{#45531}
Committed: https://chromium.googlesource.com/v8/v8/+/601f1fdf8d4b9e527ab1e5c061803ef548ecfd93

[ivica.bogosavljevic, 2017-05-25 13:47:41]

ivica.bogosavljevic@imgtec.com changed reviewers:
+ dusan.simicic@imgtec.com, jaroslav.gorin@gmail.com, miran.karic@imgtec.com,
neis@chromium.org

[ivica.bogosavljevic, 2017-05-25 13:47:42]

This fixes issue with the failure of some 400 tests. PTAL

[ivica.bogosavljevic, 2017-05-25 13:47:58]

The CQ bit was checked by ivica.bogosavljevic@imgtec.com to run a CQ dry run

[commit-bot: I haz the power, 2017-05-25 13:48:06]

Dry run: CQ is trying da patch.

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

[ivica.bogosavljevic, 2017-05-25 13:52:57]

https://codereview.chromium.org/2901393003/diff/1/src/mips/deoptimizer-mips.cc
File src/mips/deoptimizer-mips.cc (left):

https://codereview.chromium.org/2901393003/diff/1/src/mips/deoptimizer-mips.c...
src/mips/deoptimizer-mips.cc:368: __ BranchShort(USE_DELAY_SLOT, &done_special);
the issue appears here. we use the same branch to load value and as a
trampoline, but when it is used as trampoline, the original value is overwritten

[commit-bot: I haz the power, 2017-05-25 14:12:17]

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

[commit-bot: I haz the power, 2017-05-25 14:12:19]

Dry run: This issue passed the CQ dry run.

[dusan.simicic, 2017-05-25 14:18:22]

LGTM

[ivica.bogosavljevic, 2017-05-25 14:29:24]

The CQ bit was checked by ivica.bogosavljevic@imgtec.com

[commit-bot: I haz the power, 2017-05-25 14:29:27]

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-05-25 14:29:28]

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

[commit-bot: I haz the power, 2017-05-25 14:29:29]

No L-G-T-M from a valid reviewer yet.
CQ run can only be started once the patch has received an L-G-T-M from a full
committer.
Even if an L-G-T-M may have been provided, it was from a non-committer,_not_ a
full super star committer.
Committers are members of the group "project-v8-committers".
Note that this has nothing to do with OWNERS files.

[miran.karic, 2017-05-25 14:41:28]

miran.karic@imgtec.com changed reviewers:
+ Miran.Karic@imgtec.com

[miran.karic, 2017-05-25 14:41:28]

lgtm

[ivica.bogosavljevic, 2017-05-25 14:48:23]

The CQ bit was checked by ivica.bogosavljevic@imgtec.com

[commit-bot: I haz the power, 2017-05-25 14:48:27]

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-05-25 14:52:27]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1495723703576350, "parent_rev":
"7a9cc70492e77cf8754c82cb4b9a481be50c0104", "commit_rev":
"601f1fdf8d4b9e527ab1e5c061803ef548ecfd93"}

[commit-bot: I haz the power, 2017-05-25 14:52:35]

Description was changed from

==========
MIPS[64]: Fix deoptimizer generate table for large deoptimization tables

Fix 233b6926e3ae29a4567b4c55bc1c1e96c07c1f92

With the introduction of upfront deopt table generation, case where
deoptimization tables have more than 8000 entries become much more common
and bugs started to appear.

This CL fixes the issue, but it is far from perfect. We had to
increase number of instruction per table entry from 2 to 3 in
order to accommodate larger tables.

TEST=mjsunit/array-sort,mjsunit/json,mjsunit/md5
BUG=
==========

to

==========
MIPS[64]: Fix deoptimizer generate table for large deoptimization tables

Fix 233b6926e3ae29a4567b4c55bc1c1e96c07c1f92

With the introduction of upfront deopt table generation, case where
deoptimization tables have more than 8000 entries become much more common
and bugs started to appear.

This CL fixes the issue, but it is far from perfect. We had to
increase number of instruction per table entry from 2 to 3 in
order to accommodate larger tables.

TEST=mjsunit/array-sort,mjsunit/json,mjsunit/md5
BUG=

Review-Url: https://codereview.chromium.org/2901393003
Cr-Commit-Position: refs/heads/master@{#45531}
Committed:
https://chromium.googlesource.com/v8/v8/+/601f1fdf8d4b9e527ab1e5c061803ef548e...
==========

[commit-bot: I haz the power, 2017-05-25 14:52:36]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/v8/v8/+/601f1fdf8d4b9e527ab1e5c061803ef548e...