PreParsing inner funcs: declare arrow function parameters.

PreParsing inner funcs: declare arrow function parameters.

This makes the context allocation less pessimistic in case of name
clash.

This is also required for being able to skip inner functions.

BUG=v8:5501, v8:5516
Review-Url: https://codereview.chromium.org/2610823003
Cr-Commit-Position: refs/heads/master@{#42087}
Committed: https://chromium.googlesource.com/v8/v8/+/8bb2501e7f8d2575d02e3a7e2fab8c431045b838

[marja, 2017-01-04 14:08:02]

marja@chromium.org changed reviewers:
+ adamk@chromium.org

[marja, 2017-01-04 14:08:02]

adamk, ptal

[marja, 2017-01-04 14:08:08]

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

[commit-bot: I haz the power, 2017-01-04 14:08:17]

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-04 14:33:05]

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

[commit-bot: I haz the power, 2017-01-04 14:33:06]

Dry run: This issue passed the CQ dry run.

[adamk, 2017-01-04 19:28:47]

This lgtm, but I don't understand how this gets all the tests to pass. Feel free
to land, but I'd love to understand how this works.

https://codereview.chromium.org/2610823003/diff/1/test/cctest/test-parsing.cc
File test/cctest/test-parsing.cc (right):

https://codereview.chromium.org/2610823003/diff/1/test/cctest/test-parsing.cc...
test/cctest/test-parsing.cc:8654: {"function inner() { my_var => my_var; }",
false},
How does this one work?

https://codereview.chromium.org/2610823003/diff/1/test/cctest/test-parsing.cc...
test/cctest/test-parsing.cc:8668: {"function inner() { ({my_var}) => my_var;}",
false},
And this one?

[marja, 2017-01-05 09:36:20]

The CQ bit was checked by marja@chromium.org

[commit-bot: I haz the power, 2017-01-05 09:36:28]

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-05 09:38:04]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1483608980402630, "parent_rev":
"4692d96f744c5d4a4e9029ddeae2f0d31a14459c", "commit_rev":
"8bb2501e7f8d2575d02e3a7e2fab8c431045b838"}

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

Description was changed from

==========
PreParsing inner funcs: declare arrow function parameters.

This makes the context allocation less pessimistic in case of name
clash.

This is also required for being able to skip inner functions.

BUG=v8:5501, v8:5516
==========

to

==========
PreParsing inner funcs: declare arrow function parameters.

This makes the context allocation less pessimistic in case of name
clash.

This is also required for being able to skip inner functions.

BUG=v8:5501, v8:5516

Review-Url: https://codereview.chromium.org/2610823003
Cr-Commit-Position: refs/heads/master@{#42087}
Committed:
https://chromium.googlesource.com/v8/v8/+/8bb2501e7f8d2575d02e3a7e2fab8c43104...
==========

[commit-bot: I haz the power, 2017-01-05 09:38:10]

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