PPC/s390: [TypeFeedbackVector] DeclareGlobals needs a literals array

PPC/s390: [TypeFeedbackVector] DeclareGlobals needs a literals array

Port b83c3d29a95f672d16ac61b50484120e3a676d9e

Original Commit Message:

    When a function is declared in global scope, the closure is created
    by the DeclareGlobals runtime service. It needs a pointer to the
    literals array, already allocated in the feedback vector. This
    fixes a bug where it's behavior wasn't in sync with CreateClosure,
    which accepts the literals from the vector.

    This enables a follow-on performance improvement in the CompileLazy
    builtin.

R=mvstanton@chromium.org, joransiu@ca.ibm.com, jyan@ca.ibm.com, michael_dawson@ca.ibm.com
BUG=680637
LOG=N

Review-Url: https://codereview.chromium.org/2634423002
Cr-Commit-Position: refs/heads/master@{#42421}
Committed: https://chromium.googlesource.com/v8/v8/+/6555a3c1c2ef751d326f655d524e31dd8dbb8a6b

[JaideepBajwa, 2017-01-17 16:37:19]

ptal

[JoranSiu, 2017-01-17 16:41:17]

On 2017/01/17 16:37:19, JaideepBajwa wrote:
> ptal

lgtm

[JaideepBajwa, 2017-01-17 16:42:27]

The CQ bit was checked by bjaideep@ca.ibm.com

[commit-bot: I haz the power, 2017-01-17 16:42:35]

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-01-17 17:04:08]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1484671347893050, "parent_rev":
"5dbc1ba0f999161bb10548eba78aa9d22199d4be", "commit_rev":
"6555a3c1c2ef751d326f655d524e31dd8dbb8a6b"}

[commit-bot: I haz the power, 2017-01-17 17:04:12]

Description was changed from

==========
PPC/s390: [TypeFeedbackVector] DeclareGlobals needs a literals array

Port b83c3d29a95f672d16ac61b50484120e3a676d9e

Original Commit Message:

    When a function is declared in global scope, the closure is created
    by the DeclareGlobals runtime service. It needs a pointer to the
    literals array, already allocated in the feedback vector. This
    fixes a bug where it's behavior wasn't in sync with CreateClosure,
    which accepts the literals from the vector.

    This enables a follow-on performance improvement in the CompileLazy
    builtin.

R=mvstanton@chromium.org, joransiu@ca.ibm.com, jyan@ca.ibm.com,
michael_dawson@ca.ibm.com
BUG=680637
LOG=N
==========

to

==========
PPC/s390: [TypeFeedbackVector] DeclareGlobals needs a literals array

Port b83c3d29a95f672d16ac61b50484120e3a676d9e

Original Commit Message:

    When a function is declared in global scope, the closure is created
    by the DeclareGlobals runtime service. It needs a pointer to the
    literals array, already allocated in the feedback vector. This
    fixes a bug where it's behavior wasn't in sync with CreateClosure,
    which accepts the literals from the vector.

    This enables a follow-on performance improvement in the CompileLazy
    builtin.

R=mvstanton@chromium.org, joransiu@ca.ibm.com, jyan@ca.ibm.com,
michael_dawson@ca.ibm.com
BUG=680637
LOG=N

Review-Url: https://codereview.chromium.org/2634423002
Cr-Commit-Position: refs/heads/master@{#42421}
Committed:
https://chromium.googlesource.com/v8/v8/+/6555a3c1c2ef751d326f655d524e31dd8db...
==========

[commit-bot: I haz the power, 2017-01-17 17:04:13]

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