[runtime] %GrowArrayElements doesn't have a native context in TurboFan.

[runtime] %GrowArrayElements doesn't have a native context in TurboFan.

When we compile a growing store in TurboFan, we don't pass a (native)
context to the %GrowArrayElements fallback function, as the whole logic
is actually context independent. However, that means that we need to
bailout early in case the object is a prototype, which requires context
dependent checks in the array protector code.

R=cbruni@chromium.org
BUG=chromium:635798
Committed: https://crrev.com/78727d4362f76cad0e384bdf809d59fba4895195
Cr-Commit-Position: refs/heads/master@{#38491}

[Benedikt Meurer, 2016-08-09 11:53:13]

The CQ bit was checked by bmeurer@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-08-09 11:53:16]

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

[Camillo Bruni, 2016-08-09 11:53:46]

lgtm

[Benedikt Meurer, 2016-08-09 11:54:18]

The CQ bit was unchecked by bmeurer@chromium.org

[Benedikt Meurer, 2016-08-09 11:54:22]

The CQ bit was checked by bmeurer@chromium.org

[commit-bot: I haz the power, 2016-08-09 11:54: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, 2016-08-09 13:02:58]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-09 13:03:20]

Description was changed from

==========
[runtime] %GrowArrayElements doesn't have a native context in TurboFan.

When we compile a growing store in TurboFan, we don't pass a (native)
context to the %GrowArrayElements fallback function, as the whole logic
is actually context independent. However, that means that we need to
bailout early in case the object is a prototype, which requires context
dependent checks in the array protector code.

R=cbruni@chromium.org
BUG=chromium:635798
==========

to

==========
[runtime] %GrowArrayElements doesn't have a native context in TurboFan.

When we compile a growing store in TurboFan, we don't pass a (native)
context to the %GrowArrayElements fallback function, as the whole logic
is actually context independent. However, that means that we need to
bailout early in case the object is a prototype, which requires context
dependent checks in the array protector code.

R=cbruni@chromium.org
BUG=chromium:635798

Committed: https://crrev.com/78727d4362f76cad0e384bdf809d59fba4895195
Cr-Commit-Position: refs/heads/master@{#38491}
==========

[commit-bot: I haz the power, 2016-08-09 13:03:20]

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

[Michael Achenbach, 2016-08-11 10:04:31]

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

[Michael Achenbach, 2016-08-11 10:04:32]

The test kills arm64 gc stress:
https://build.chromium.org/p/client.v8.ports/builders/V8%20Linux%20-%20arm64%...

The test takes too long and lets the whole driver time out.

Could you skip it for arm64 and gcstress or speed it up somehow?

[Michael Achenbach, 2016-08-29 06:26:44]

Btw: This is also the reason why our arm64 sim debug is purple most of the time.
Test times:
https://build.chromium.org/p/client.v8.ports/builders/V8%20Linux%20-%20arm64%...

This is way too slow. If the test can't be sped up, I'm going to skip it.