MIPS64: Fix [runtime] Introduce dedicated JSBoundFunction to represent bound functions.

MIPS64: Fix [runtime] Introduce dedicated JSBoundFunction to represent bound functions.

Fix 97def8070cbb979419c041c3732d5e11779f79da

On MIPS64, the address of BoundTargetFunction was calculated badly
because instruction ADD was used for this instead of instruction DADD.
This caused flaky crashes of several tests.

TEST=mjsunit/es6/debug-step-into-constructor,mjsunit/es6/array-of,
mjsunit/function-bind,...

BUG=

Committed: https://crrev.com/7b42c6cfc665a81b903ffed709883121b3acee43
Cr-Commit-Position: refs/heads/master@{#33048}

[ivica.bogosavljevic, 2015-12-28 13:45:28]

ivica.bogosavljevic@imgtec.com changed reviewers:
+ akos.palfi@imgtec.com, ilija.pavlovic@imgtec.com, miran.karic@imgtec.com,
paul.lind@imgtec.com

[ivica.bogosavljevic, 2015-12-28 13:45:29]

PTAL

[ivica.bogosavljevic, 2015-12-28 13:46:30]

https://codereview.chromium.org/1549383002/diff/1/src/mips64/builtins-mips64.cc
File src/mips64/builtins-mips64.cc (right):

https://codereview.chromium.org/1549383002/diff/1/src/mips64/builtins-mips64....
src/mips64/builtins-mips64.cc:2179: __ Daddu(at, at, Operand(Code::kHeaderSize -
kHeapObjectTag));
Full list of failures:
https://v8.mips.com/bbv8/builders/Sim%20-%20MIPS64%20-%20Debug%20-%20edge%20-...

[ivica.bogosavljevic, 2015-12-28 13:48:36]

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

[commit-bot: I haz the power, 2015-12-28 13:48:51]

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

[commit-bot: I haz the power, 2015-12-28 13:48:52]

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

[commit-bot: I haz the power, 2015-12-28 13:48:52]

Dry run: No L-G-T-M from a valid reviewer yet. Only full committers are
accepted.
Even if an L-G-T-M may have been provided, it was from a non-committer,
_not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[paul.l..., 2015-12-28 14:52:18]

paul.lind@imgtec.com changed reviewers:
+ hablich@chromium.org, machenbach@chromium.org

[paul.l..., 2015-12-28 14:52:19]

LGTM. Please do retry the CQ dry-run, per separate email I just sent. Feel free
to commit once the dry-run finishes.

[ivica.bogosavljevic, 2015-12-28 15:03:19]

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

[commit-bot: I haz the power, 2015-12-28 15:03:23]

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

[ivica.bogosavljevic, 2015-12-28 15:04:44]

On 2015/12/28 14:52:19, paul.l... wrote:
> LGTM. Please do retry the CQ dry-run, per separate email I just sent. Feel
free
> to commit once the dry-run finishes.

I added the v8_android_arm_compile_rel  bot, clicked dry run and this time it
worked for me.

[commit-bot: I haz the power, 2015-12-28 15:24:51]

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

[commit-bot: I haz the power, 2015-12-28 15:24:52]

Dry run: This issue passed the CQ dry run.

[ivica.bogosavljevic, 2015-12-28 15:25:12]

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

[commit-bot: I haz the power, 2015-12-28 15:25:19]

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

[commit-bot: I haz the power, 2015-12-28 15:26:39]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-12-28 15:27:27]

Description was changed from

==========
MIPS64: Fix [runtime] Introduce dedicated JSBoundFunction to represent bound
functions.

Fix 97def8070cbb979419c041c3732d5e11779f79da

On MIPS64, the address of BoundTargetFunction was calculated badly
because instruction ADD was used for this instead of instruction DADD.
This caused flaky crashes of several tests.

TEST=mjsunit/es6/debug-step-into-constructor,mjsunit/es6/array-of,
mjsunit/function-bind,...

BUG=
==========

to

==========
MIPS64: Fix [runtime] Introduce dedicated JSBoundFunction to represent bound
functions.

Fix 97def8070cbb979419c041c3732d5e11779f79da

On MIPS64, the address of BoundTargetFunction was calculated badly
because instruction ADD was used for this instead of instruction DADD.
This caused flaky crashes of several tests.

TEST=mjsunit/es6/debug-step-into-constructor,mjsunit/es6/array-of,
mjsunit/function-bind,...

BUG=

Committed: https://crrev.com/7b42c6cfc665a81b903ffed709883121b3acee43
Cr-Commit-Position: refs/heads/master@{#33048}
==========

[commit-bot: I haz the power, 2015-12-28 15:27:29]

Patchset 1 (id:??) landed as
https://crrev.com/7b42c6cfc665a81b903ffed709883121b3acee43
Cr-Commit-Position: refs/heads/master@{#33048}