[ast] Never declare non-lexical variables as kNeedsInitialization.

[ast] Never declare non-lexical variables as kNeedsInitialization.

R=adamk@chromium.org
BUG=

Committed: https://crrev.com/63f311ad3493344b3d12d63ec9537ebbe1a714d0
Cr-Commit-Position: refs/heads/master@{#38589}

[neis, 2016-08-11 17:17:33]

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

[commit-bot: I haz the power, 2016-08-11 17:17:42]

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

[adamk, 2016-08-11 17:17:48]

lgtm

The truth of the matter is we can't tell when we have a dynamic variable whether
it needs hole checks or not. We only find out when we resolve it at runtime, and
the hole check is done then:

https://cs.chromium.org/chromium/src/v8/src/runtime/runtime-scopes.cc?rcl=0&l...

[neis, 2016-08-11 17:19:51]

On 2016/08/11 17:17:48, adamk wrote:
> lgtm
> 
> The truth of the matter is we can't tell when we have a dynamic variable
whether
> it needs hole checks or not. We only find out when we resolve it at runtime,
and
> the hole check is done then:
> 
>
https://cs.chromium.org/chromium/src/v8/src/runtime/runtime-scopes.cc?rcl=0&l...

Thanks for the pointer.

[commit-bot: I haz the power, 2016-08-11 17:40:39]

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

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

Dry run: This issue passed the CQ dry run.

[neis, 2016-08-11 17:43:01]

The CQ bit was checked by neis@chromium.org

[commit-bot: I haz the power, 2016-08-11 17:43:10]

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-11 17:44:55]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-11 17:45:06]

Description was changed from

==========
[ast] Never declare non-lexical variables as kNeedsInitialization.

R=adamk@chromium.org
BUG=
==========

to

==========
[ast] Never declare non-lexical variables as kNeedsInitialization.

R=adamk@chromium.org
BUG=

Committed: https://crrev.com/63f311ad3493344b3d12d63ec9537ebbe1a714d0
Cr-Commit-Position: refs/heads/master@{#38589}
==========

[commit-bot: I haz the power, 2016-08-11 17:45:08]

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