[FeedbackVector] Infrastructure for literal arrays in the vector.

[FeedbackVector] Infrastructure for literal arrays in the vector.

This changes the NewClosure interface descriptor, but ignores
the additional vector/slot arguments for now. The feedback vector
gets larger, as it holds a space for each literal array. A follow-on
CL will constructively use this space.

BUG=v8:5456
Review-Url: https://codereview.chromium.org/2614373002
Cr-Commit-Position: refs/heads/master@{#42146}
Committed: https://chromium.googlesource.com/v8/v8/+/38602f1ff553b32815d34a05cd6d965f4fc20960

[mvstanton, 2017-01-09 12:38:21]

Description was changed from

==========
[FeedbackVector] Infrastructure for literal arrays in the vector.

This changes the NewClosure interface descriptor, but ignores
the additional vector/slot arguments for now.

BUG=v8:5456
==========

to

==========
[FeedbackVector] Infrastructure for literal arrays in the vector.

This changes the NewClosure interface descriptor, but ignores
the additional vector/slot arguments for now. The feedback vector
gets larger, as it holds a space for each literal array. A follow-on
CL will constructively use this space.

BUG=v8:5456
==========

[mvstanton, 2017-01-09 12:38:21]

mvstanton@chromium.org changed reviewers:
+ bmeurer@chromium.org

[mvstanton, 2017-01-09 13:05:49]

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

[commit-bot: I haz the power, 2017-01-09 13:05:53]

Dry run: 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-09 13:07:25]

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

[commit-bot: I haz the power, 2017-01-09 13:07:26]

Dry run: Try jobs failed on following builders:
  v8_android_arm_compile_rel on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_android_arm_compile_rel/...)
  v8_linux64_avx2_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_avx2_rel_ng/buil...)
  v8_linux64_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_rel_ng/builds/18745)
  v8_linux_arm64_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_arm64_rel_ng/build...)
  v8_linux_arm_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_arm_rel_ng/builds/...)
  v8_linux_gcc_compile_rel on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_gcc_compile_rel/bu...)
  v8_linux_nodcheck_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_nodcheck_rel_ng/bu...)
  v8_presubmit on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_presubmit/builds/31843)

[mvstanton, 2017-01-09 13:08:04]

Hi Benedikt,

Here is the *partial* re-land of the literal array change we've discussed. This
CL contains infrastructure, reserving changes to object structure for the next
one. This piece should be able to remain in the tree as we get closer.
Thanks,
--Michael

[Benedikt Meurer, 2017-01-09 13:46:14]

LGTM.

[mvstanton, 2017-01-09 14:16:20]

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

[commit-bot: I haz the power, 2017-01-09 14:16:32]

Dry run: 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-09 14:33:37]

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

[commit-bot: I haz the power, 2017-01-09 14:33:38]

Dry run: Try jobs failed on following builders:
  v8_linux64_gyp_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_gyp_rel_ng/build...)

[mvstanton, 2017-01-09 14:51:21]

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

[commit-bot: I haz the power, 2017-01-09 14:51:28]

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

[mvstanton, 2017-01-09 14:51:32]

The CQ bit was checked by mvstanton@chromium.org

[mvstanton, 2017-01-09 14:51:33]

The patchset sent to the CQ was uploaded after l-g-t-m from bmeurer@chromium.org
Link to the patchset: https://codereview.chromium.org/2614373002/#ps60001
(title: "Release compile fix.")

[commit-bot: I haz the power, 2017-01-09 14:51:38]

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-09 15:31:02]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1483973492581550,
"parent_rev": "e7fc42796c823fdee6b4f812f798679df42fba0c", "commit_rev":
"38602f1ff553b32815d34a05cd6d965f4fc20960"}

[commit-bot: I haz the power, 2017-01-09 15:31:05]

Description was changed from

==========
[FeedbackVector] Infrastructure for literal arrays in the vector.

This changes the NewClosure interface descriptor, but ignores
the additional vector/slot arguments for now. The feedback vector
gets larger, as it holds a space for each literal array. A follow-on
CL will constructively use this space.

BUG=v8:5456
==========

to

==========
[FeedbackVector] Infrastructure for literal arrays in the vector.

This changes the NewClosure interface descriptor, but ignores
the additional vector/slot arguments for now. The feedback vector
gets larger, as it holds a space for each literal array. A follow-on
CL will constructively use this space.

BUG=v8:5456

Review-Url: https://codereview.chromium.org/2614373002
Cr-Commit-Position: refs/heads/master@{#42146}
Committed:
https://chromium.googlesource.com/v8/v8/+/38602f1ff553b32815d34a05cd6d965f4fc...
==========

[commit-bot: I haz the power, 2017-01-09 15:31:06]

Committed patchset #4 (id:60001) as
https://chromium.googlesource.com/v8/v8/+/38602f1ff553b32815d34a05cd6d965f4fc...