[ic] Pass name to LoadGlobalIC again.

[ic] Pass name to LoadGlobalIC again.

The reasons are:
1) The names dictionaries in the feedback metadata seems to consume a lot of memory
   and the idea didn't payoff.
2) The absence of a name parameter blocks data handlers support in LoadGlobalIC.

This CL reverts a part of r37278 (https://codereview.chromium.org/2096653003/).

BUG=chromium:576312, v8:5561
Committed: https://crrev.com/d3231f5144d4d2b792e54055ba17fbbb347caa55
Cr-Commit-Position: refs/heads/master@{#41046}

[Igor Sheludko, 2016-11-16 11:46:08]

Patchset #1 (id:1) has been deleted

[Igor Sheludko, 2016-11-16 11:46:34]

Description was changed from

==========
[ic] Pass name to LoadGlobalIC again.

The reasons are:
1) The names dictionary in the feedback metadata seems to consume a lot of
memory
   and the idea didn't payoff.
2) The absence of a name parameter blocks data handlers support in LoadGlobalIC.

This CL reverts a part of r37278 (https://codereview.chromium.org/2096653003/).

BUG=chromium:576312, v8:5561
==========

to

==========
[ic] Pass name to LoadGlobalIC again.

The reasons are:
1) The names dictionaries in the feedback metadata seems to consume a lot of
memory
   and the idea didn't payoff.
2) The absence of a name parameter blocks data handlers support in LoadGlobalIC.

This CL reverts a part of r37278 (https://codereview.chromium.org/2096653003/).

BUG=chromium:576312, v8:5561
==========

[Igor Sheludko, 2016-11-16 11:48:12]

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

[commit-bot: I haz the power, 2016-11-16 11:48:19]

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

[Igor Sheludko, 2016-11-16 11:49:09]

ishell@chromium.org changed reviewers:
+ jkummerow@chromium.org, rmcilroy@chromium.org

[Igor Sheludko, 2016-11-16 11:49:10]

Ross, PTAL interpreter
Jakob, PTAL the rest

[commit-bot: I haz the power, 2016-11-16 11:49:46]

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

[commit-bot: I haz the power, 2016-11-16 11:49:47]

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_asan_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_asan_rel_ng/buil...)
  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_gyp_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_gyp_rel_ng/build...)
  v8_linux64_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_rel_ng/builds/16285)
  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_dbg_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_dbg_ng/builds/16326)
  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_mips64el_compile_rel on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_mips64el_compile_r...)
  v8_linux_mipsel_compile_rel on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_mipsel_compile_rel...)
  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_linux_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_rel_ng/builds/16247)
  v8_presubmit on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_presubmit/builds/28654)
  v8_win_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_win_rel_ng/builds/17952)

[rmcilroy, 2016-11-16 12:52:53]

interpreter/ LGTM.

[Jakob Kummerow, 2016-11-16 13:54:25]

LGTM % rebasing.

You'll also need src/compiler/OWNERS review.

[Benedikt Meurer, 2016-11-16 14:00:02]

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

[Benedikt Meurer, 2016-11-16 14:00:03]

LGTM on compiler.

[Igor Sheludko, 2016-11-16 17:37:19]

The CQ bit was checked by ishell@chromium.org

[Igor Sheludko, 2016-11-16 17:37:20]

The patchset sent to the CQ was uploaded after l-g-t-m from
bmeurer@chromium.org, rmcilroy@chromium.org, jkummerow@chromium.org
Link to the patchset: https://codereview.chromium.org/2510653002/#ps40001
(title: "Rebasing")

[commit-bot: I haz the power, 2016-11-16 17:37:25]

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-11-16 17:40:53]

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

[commit-bot: I haz the power, 2016-11-16 17:40:54]

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_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_rel_ng/builds/16335)
  v8_linux_mips64el_compile_rel on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_mips64el_compile_r...)
  v8_linux_mipsel_compile_rel on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_mipsel_compile_rel...)

[Igor Sheludko, 2016-11-16 17:47:30]

The CQ bit was checked by ishell@chromium.org

[Igor Sheludko, 2016-11-16 17:47:31]

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

[commit-bot: I haz the power, 2016-11-16 17:47:34]

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-11-16 18:17:29]

Description was changed from

==========
[ic] Pass name to LoadGlobalIC again.

The reasons are:
1) The names dictionaries in the feedback metadata seems to consume a lot of
memory
   and the idea didn't payoff.
2) The absence of a name parameter blocks data handlers support in LoadGlobalIC.

This CL reverts a part of r37278 (https://codereview.chromium.org/2096653003/).

BUG=chromium:576312, v8:5561
==========

to

==========
[ic] Pass name to LoadGlobalIC again.

The reasons are:
1) The names dictionaries in the feedback metadata seems to consume a lot of
memory
   and the idea didn't payoff.
2) The absence of a name parameter blocks data handlers support in LoadGlobalIC.

This CL reverts a part of r37278 (https://codereview.chromium.org/2096653003/).

BUG=chromium:576312, v8:5561
==========

[commit-bot: I haz the power, 2016-11-16 18:17:30]

Committed patchset #3 (id:60001)

[Benedikt Meurer, 2016-11-17 04:38:54]

For the record, this tanks SunSpider by ~2% w/ Ignition+TurboFan. I guess that's
an indicator we're still doing it something wrong in that pipeline.

[commit-bot: I haz the power, 2016-11-17 22:36:50]

Description was changed from

==========
[ic] Pass name to LoadGlobalIC again.

The reasons are:
1) The names dictionaries in the feedback metadata seems to consume a lot of
memory
   and the idea didn't payoff.
2) The absence of a name parameter blocks data handlers support in LoadGlobalIC.

This CL reverts a part of r37278 (https://codereview.chromium.org/2096653003/).

BUG=chromium:576312, v8:5561
==========

to

==========
[ic] Pass name to LoadGlobalIC again.

The reasons are:
1) The names dictionaries in the feedback metadata seems to consume a lot of
memory
   and the idea didn't payoff.
2) The absence of a name parameter blocks data handlers support in LoadGlobalIC.

This CL reverts a part of r37278 (https://codereview.chromium.org/2096653003/).

BUG=chromium:576312, v8:5561

Committed: https://crrev.com/d3231f5144d4d2b792e54055ba17fbbb347caa55
Cr-Commit-Position: refs/heads/master@{#41046}
==========

[commit-bot: I haz the power, 2016-11-17 22:36:51]

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