X87: Use Variable::binding_needs_init() to determine hole initialization.

X87: Use Variable::binding_needs_init() to determine hole initialization.

  port 6768456db513936cb19c8cdc657a7cd6122584a9 (r38395)

  original commit message:
  The old code was using VariableMode, but that signal is both
  over-pessimistic (some CONST and LET variables need no hole-initialization)
  and inconsistent with other uses of the InitializationFlag enum (such
  as %LoadLookupSlot).

  This changes no observable behavior, but removes unnecessary hole
  initialization and hole checks in a few places, including
  block-scoped function declarations, super property lookups,
  and new.target.

BUG=

Committed: https://crrev.com/759a06fca37a21dd62a21fbad602c3fb874c495c
Cr-Commit-Position: refs/heads/master@{#38468}

[zhengxing.li, 2016-08-08 09:38:30]

zhengxing.li@intel.com changed reviewers:
+ adamk@chromium.org, chunyang.dai@intel.com, weiliang.lin@intel.com

[zhengxing.li, 2016-08-08 09:38:30]

PTAL, thanks!

[Weiliang, 2016-08-09 02:36:53]

lgtm

[zhengxing.li, 2016-08-09 03:09:26]

The CQ bit was checked by zhengxing.li@intel.com

[commit-bot: I haz the power, 2016-08-09 03:09:36]

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-08-09 06:49:17]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-09 06:53:15]

Description was changed from

==========
X87: Use Variable::binding_needs_init() to determine hole initialization.

  port 6768456db513936cb19c8cdc657a7cd6122584a9 (r38395)

  original commit message:
  The old code was using VariableMode, but that signal is both
  over-pessimistic (some CONST and LET variables need no hole-initialization)
  and inconsistent with other uses of the InitializationFlag enum (such
  as %LoadLookupSlot).

  This changes no observable behavior, but removes unnecessary hole
  initialization and hole checks in a few places, including
  block-scoped function declarations, super property lookups,
  and new.target.

BUG=
==========

to

==========
X87: Use Variable::binding_needs_init() to determine hole initialization.

  port 6768456db513936cb19c8cdc657a7cd6122584a9 (r38395)

  original commit message:
  The old code was using VariableMode, but that signal is both
  over-pessimistic (some CONST and LET variables need no hole-initialization)
  and inconsistent with other uses of the InitializationFlag enum (such
  as %LoadLookupSlot).

  This changes no observable behavior, but removes unnecessary hole
  initialization and hole checks in a few places, including
  block-scoped function declarations, super property lookups,
  and new.target.

BUG=

Committed: https://crrev.com/759a06fca37a21dd62a21fbad602c3fb874c495c
Cr-Commit-Position: refs/heads/master@{#38468}
==========

[commit-bot: I haz the power, 2016-08-09 06:53:16]

Patchset 1 (id:??) landed as
https://crrev.com/759a06fca37a21dd62a21fbad602c3fb874c495c
Cr-Commit-Position: refs/heads/master@{#38468}