Improve strictness of Annex B 3.3 for generators and async functions

Improve strictness of Annex B 3.3 for generators and async functions

Annex B 3.3 applies only for ordinary FunctionDeclarations, not
GeneratorDeclarations or AsyncFunctionDeclarations. This patch
- Skips applying Annex B 3.3 to async functions
- Adds a flag to refrain from applying it to generators
- UseCounter for how often duplicate function in block occurs
  with generators (unclear how to measure need for hoisting from block)

BUG=v8:4806
Committed: https://crrev.com/6390282f9654dcefe1d2123298b50db9ad5ce1f6
Cr-Commit-Position: refs/heads/master@{#36557}

[Dan Ehrenberg, 2016-05-19 00:11:47]

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

[commit-bot: I haz the power, 2016-05-19 00:11:58]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1995863002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1995863002/1

[Dan Ehrenberg, 2016-05-19 00:12:13]

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

[commit-bot: I haz the power, 2016-05-19 00:15:08]

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

[commit-bot: I haz the power, 2016-05-19 00:15:09]

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

[Dan Ehrenberg, 2016-05-19 00:20:41]

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

[commit-bot: I haz the power, 2016-05-19 00:20:48]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1995863002/10001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1995863002/10001

[commit-bot: I haz the power, 2016-05-19 00:22:58]

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

[commit-bot: I haz the power, 2016-05-19 00:22:59]

Dry run: Try jobs failed on following builders:
  v8_linux64_asan_rel_ng on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_asan_rel_ng/buil...)
  v8_linux_arm64_rel_ng on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_arm64_rel_ng/build...)
  v8_linux_arm_rel_ng on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_arm_rel_ng/builds/...)
  v8_linux_mipsel_compile_rel on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_mipsel_compile_rel...)
  v8_linux_rel_ng on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_rel_ng/builds/5936)
  v8_mac_rel_ng on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_mac_rel_ng/builds/1843)

[Dan Ehrenberg, 2016-05-19 01:26:08]

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

[commit-bot: I haz the power, 2016-05-19 01:26:12]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1995863002/30001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1995863002/30001

[commit-bot: I haz the power, 2016-05-19 01:31:54]

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

[commit-bot: I haz the power, 2016-05-19 01:31:55]

Dry run: Try jobs failed on following builders:
  v8_linux_arm_rel_ng on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_arm_rel_ng/builds/...)
  v8_linux_arm_rel_ng_triggered on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_arm_rel_ng_trigger...)

[adamk, 2016-05-20 20:49:11]

Looks like you've got some test failures? Ping back when those are resolved,
please.

[Dan Ehrenberg, 2016-05-27 16:46:33]

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

[commit-bot: I haz the power, 2016-05-27 16:46:38]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1995863002/50001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1995863002/50001

[commit-bot: I haz the power, 2016-05-27 17:03:18]

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

[commit-bot: I haz the power, 2016-05-27 17:03:20]

Dry run: Try jobs failed on following builders:
  v8_linux64_asan_rel_ng on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_asan_rel_ng/buil...)
  v8_linux64_asan_rel_ng_triggered on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_asan_rel_ng_trig...)
  v8_linux64_avx2_rel_ng on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_avx2_rel_ng/buil...)
  v8_linux64_avx2_rel_ng_triggered on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_avx2_rel_ng_trig...)

[Dan Ehrenberg, 2016-05-27 17:42:20]

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

[commit-bot: I haz the power, 2016-05-27 17:42:29]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1995863002/70001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1995863002/70001

[commit-bot: I haz the power, 2016-05-27 18:05:57]

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

[commit-bot: I haz the power, 2016-05-27 18:05:58]

Dry run: This issue passed the CQ dry run.

[Dan Ehrenberg, 2016-05-27 18:08:52]

The bugs are fixed. PTAL.

[adamk, 2016-05-27 18:18:12]

lgtm

[Dan Ehrenberg, 2016-05-27 18:20:23]

The CQ bit was checked by littledan@chromium.org

[commit-bot: I haz the power, 2016-05-27 18:20:28]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1995863002/70001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1995863002/70001

[commit-bot: I haz the power, 2016-05-27 18:22:39]

Committed patchset #5 (id:70001)

[commit-bot: I haz the power, 2016-05-27 18:23:26]

Description was changed from

==========
Improve strictness of Annex B 3.3 for generators and async functions

Annex B 3.3 applies only for ordinary FunctionDeclarations, not
GeneratorDeclarations or AsyncFunctionDeclarations. This patch
- Skips applying Annex B 3.3 to async functions
- Adds a flag to refrain from applying it to generators
- UseCounter for how often duplicate function in block occurs
  with generators (unclear how to measure need for hoisting from block)

BUG=v8:4806
==========

to

==========
Improve strictness of Annex B 3.3 for generators and async functions

Annex B 3.3 applies only for ordinary FunctionDeclarations, not
GeneratorDeclarations or AsyncFunctionDeclarations. This patch
- Skips applying Annex B 3.3 to async functions
- Adds a flag to refrain from applying it to generators
- UseCounter for how often duplicate function in block occurs
  with generators (unclear how to measure need for hoisting from block)

BUG=v8:4806

Committed: https://crrev.com/6390282f9654dcefe1d2123298b50db9ad5ce1f6
Cr-Commit-Position: refs/heads/master@{#36557}
==========

[commit-bot: I haz the power, 2016-05-27 18:23:27]

Patchset 5 (id:??) landed as
https://crrev.com/6390282f9654dcefe1d2123298b50db9ad5ce1f6
Cr-Commit-Position: refs/heads/master@{#36557}