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

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

Port d287c8196990b04169d5dc4e9cf6eb03d88d9e47

Original Commit Message:

    [RELAND with one change: until literal arrays are rooted in the outer
    feedback vector (coming in the next days), the runtime-scope.cc change
    is held off.]

    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/2658053002
Cr-Commit-Position: refs/heads/master@{#42705}
Committed: https://chromium.googlesource.com/v8/v8/+/842748006dfb92b46bf2dcb2b1ac64f75c355324

[JaideepBajwa, 2017-01-26 15:09:46]

ptal

[john.yan, 2017-01-26 15:11:08]

lgtm

[JaideepBajwa, 2017-01-26 15:11:20]

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

[commit-bot: I haz the power, 2017-01-26 15:11:24]

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-26 15:43:00]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1485443480926530, "parent_rev":
"d2d99e084ad7878cd32746a40f657e51ca697a3e", "commit_rev":
"842748006dfb92b46bf2dcb2b1ac64f75c355324"}

[commit-bot: I haz the power, 2017-01-26 15:43:03]

Description was changed from

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

Port d287c8196990b04169d5dc4e9cf6eb03d88d9e47

Original Commit Message:

    [RELAND with one change: until literal arrays are rooted in the outer
    feedback vector (coming in the next days), the runtime-scope.cc change
    is held off.]

    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 d287c8196990b04169d5dc4e9cf6eb03d88d9e47

Original Commit Message:

    [RELAND with one change: until literal arrays are rooted in the outer
    feedback vector (coming in the next days), the runtime-scope.cc change
    is held off.]

    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/2658053002
Cr-Commit-Position: refs/heads/master@{#42705}
Committed:
https://chromium.googlesource.com/v8/v8/+/842748006dfb92b46bf2dcb2b1ac64f75c3...
==========

[commit-bot: I haz the power, 2017-01-26 15:43:04]

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