MIPS: Fix function Fast_ArrayConcat.

MIPS: Fix function Fast_ArrayConcat.

Port for https://codereview.chromium.org/1409123003

In function Fast_ArrayConcat is added comparation between result_len and
FixedDoubleArray::kMaxLength. This change enables successful execution
of the test regress-599414-array-concat-fast-path.

Original commit message:
  [runtime] Avoid @@isConcatSpreadable lookup for fast path Array.prototype.concat

  Currently we do not check for @@isConcatSpreadable properly. If the Symbol is
  set on the Array.prototype or Object.prototype the current fast paths fail.
  This CL adds a fix to globally invalidate a isConcatSpreadable_protector.

  Drive-by-fix: use named accessors for context variables

TEST=mjsunit/regress/regress-599414-array-concat-fast-path
BUG=

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

[Ilija.Pavlovic1, 2016-05-20 10:40:55]

ilija.pavlovic@imgtec.com changed reviewers:
+ akos.palfi@imgtec.com, balazs.kilvady@imgtec.com, cbruni@chromium.org,
gergely.kis@imgtec.com, ivica.bogosavljevic@imgtec.com, marija.antic@imgtec.com,
miran.karic@imgtec.com, ulan@chromium.org, verwaest@chromium.org

[Ilija.Pavlovic1, 2016-05-20 10:40:56]

PTAL

[balazs.kilvady, 2016-05-20 11:04:50]

lgtm. This does need a v8-team review. Can one of you PTAL?

[Camillo Bruni, 2016-05-20 13:19:11]

lgtm

[Ilija.Pavlovic1, 2016-05-23 06:34:06]

The CQ bit was checked by ilija.pavlovic@imgtec.com

[commit-bot: I haz the power, 2016-05-23 06:34:17]

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

[commit-bot: I haz the power, 2016-05-23 07:07:28]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-05-23 07:09:23]

Description was changed from

==========
MIPS: Fix function Fast_ArrayConcat.

Port for https://codereview.chromium.org/1409123003

In function Fast_ArrayConcat is added comparation between result_len and
FixedDoubleArray::kMaxLength. This change enables successful execution
of the test regress-599414-array-concat-fast-path.

Original commit message:
  [runtime] Avoid @@isConcatSpreadable lookup for fast path
Array.prototype.concat

  Currently we do not check for @@isConcatSpreadable properly. If the Symbol is
  set on the Array.prototype or Object.prototype the current fast paths fail.
  This CL adds a fix to globally invalidate a isConcatSpreadable_protector.

  Drive-by-fix: use named accessors for context variables

TEST=mjsunit/regress/regress-599414-array-concat-fast-path
BUG=
==========

to

==========
MIPS: Fix function Fast_ArrayConcat.

Port for https://codereview.chromium.org/1409123003

In function Fast_ArrayConcat is added comparation between result_len and
FixedDoubleArray::kMaxLength. This change enables successful execution
of the test regress-599414-array-concat-fast-path.

Original commit message:
  [runtime] Avoid @@isConcatSpreadable lookup for fast path
Array.prototype.concat

  Currently we do not check for @@isConcatSpreadable properly. If the Symbol is
  set on the Array.prototype or Object.prototype the current fast paths fail.
  This CL adds a fix to globally invalidate a isConcatSpreadable_protector.

  Drive-by-fix: use named accessors for context variables

TEST=mjsunit/regress/regress-599414-array-concat-fast-path
BUG=

Committed: https://crrev.com/e2e879694af027fde9c6f003a5281a135a5ba45f
Cr-Commit-Position: refs/heads/master@{#36429}
==========

[commit-bot: I haz the power, 2016-05-23 07:09:24]

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