[stubs] Add option to allow LO space allocation

[stubs] Add option to allow LO space allocation

Passing kAllowLargeObjectAllocation now allocates in LOS if necessary.
Allow such allocations when growing fixed arrays in RegExp's @@match
and @@split operations.

BUG=chromium:670671
Committed: https://crrev.com/9c9c8d7bb5681c9fe2a0c2bbe21f97f8b33d9c46
Cr-Commit-Position: refs/heads/master@{#41526}

[jgruber, 2016-12-06 12:29:57]

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

[commit-bot: I haz the power, 2016-12-06 12:30:08]

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

[jgruber, 2016-12-06 12:32:01]

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

[commit-bot: I haz the power, 2016-12-06 12:32:06]

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

[jgruber, 2016-12-06 12:44:10]

Description was changed from

==========
[stubs] Add option to allow LO space allocation

Passing kAllowLargeObjectSpace now allocates in LOS if necessary. Allow such
allocations when growing fixed arrays in RegExp's @@match and @@split
operations.

BUG=chromium:670671
==========

to

==========
[stubs] Add option to allow LO space allocation

Passing kAllowLargeObjectAllocation now allocates in LOS if necessary.
Allow such allocations when growing fixed arrays in RegExp's @@match 
and @@split operations.

BUG=chromium:670671
==========

[jgruber, 2016-12-06 12:54:26]

jgruber@chromium.org changed reviewers:
+ ishell@chromium.org

[commit-bot: I haz the power, 2016-12-06 12:58:36]

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

[commit-bot: I haz the power, 2016-12-06 12:58:37]

Dry run: This issue passed the CQ dry run.

[Igor Sheludko, 2016-12-06 13:18:05]

https://codereview.chromium.org/2555703003/diff/20001/src/runtime/runtime-int...
File src/runtime/runtime-internal.cc (right):

https://codereview.chromium.org/2555703003/diff/20001/src/runtime/runtime-int...
src/runtime/runtime-internal.cc:257:
RUNTIME_FUNCTION(Runtime_AllocateInLargeObjectSpace) {
Probably it's better to use existing Runtime_AllocateInTargetSpace instead. We
just need to update the size check there.

[jgruber, 2016-12-06 13:32:19]

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

[commit-bot: I haz the power, 2016-12-06 13:32:25]

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

[jgruber, 2016-12-06 13:33:08]

https://codereview.chromium.org/2555703003/diff/20001/src/runtime/runtime-int...
File src/runtime/runtime-internal.cc (right):

https://codereview.chromium.org/2555703003/diff/20001/src/runtime/runtime-int...
src/runtime/runtime-internal.cc:257:
RUNTIME_FUNCTION(Runtime_AllocateInLargeObjectSpace) {
On 2016/12/06 13:18:05, Igor Sheludko wrote:
> Probably it's better to use existing Runtime_AllocateInTargetSpace instead. We
> just need to update the size check there.

Done.

[Igor Sheludko, 2016-12-06 13:39:00]

lgtm

[commit-bot: I haz the power, 2016-12-06 14:00:16]

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

[commit-bot: I haz the power, 2016-12-06 14:00:17]

Dry run: This issue passed the CQ dry run.

[jgruber, 2016-12-06 14:06:53]

The CQ bit was checked by jgruber@chromium.org

[commit-bot: I haz the power, 2016-12-06 14:07:00]

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-12-06 14:08:42]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1481033213982840,
"parent_rev": "d1c46e50b9721e7c440c83f6496e651418b34940", "commit_rev":
"c166ac469d17aad52d48fbd8dc5da02864a19be7"}

[commit-bot: I haz the power, 2016-12-06 14:08:45]

Description was changed from

==========
[stubs] Add option to allow LO space allocation

Passing kAllowLargeObjectAllocation now allocates in LOS if necessary.
Allow such allocations when growing fixed arrays in RegExp's @@match 
and @@split operations.

BUG=chromium:670671
==========

to

==========
[stubs] Add option to allow LO space allocation

Passing kAllowLargeObjectAllocation now allocates in LOS if necessary.
Allow such allocations when growing fixed arrays in RegExp's @@match 
and @@split operations.

BUG=chromium:670671
==========

[commit-bot: I haz the power, 2016-12-06 14:08:46]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-12-06 14:09:03]

Description was changed from

==========
[stubs] Add option to allow LO space allocation

Passing kAllowLargeObjectAllocation now allocates in LOS if necessary.
Allow such allocations when growing fixed arrays in RegExp's @@match 
and @@split operations.

BUG=chromium:670671
==========

to

==========
[stubs] Add option to allow LO space allocation

Passing kAllowLargeObjectAllocation now allocates in LOS if necessary.
Allow such allocations when growing fixed arrays in RegExp's @@match
and @@split operations.

BUG=chromium:670671

Committed: https://crrev.com/9c9c8d7bb5681c9fe2a0c2bbe21f97f8b33d9c46
Cr-Commit-Position: refs/heads/master@{#41526}
==========

[commit-bot: I haz the power, 2016-12-06 14:09:04]

Patchset 3 (id:??) landed as
https://crrev.com/9c9c8d7bb5681c9fe2a0c2bbe21f97f8b33d9c46
Cr-Commit-Position: refs/heads/master@{#41526}