[es6] More efficient way of marking AST call expressions in tail positions.

[es6] More efficient way of marking AST call expressions in tail positions.

Instead of doing a full function body traversal we collect return expressions and mark them after function parsing.

And since we rewrite do-expressions so that the result is explicitly assigned to a result variable the statements marking will never hit so I removed it from the AST.

BUG=v8:4698
LOG=N

Committed: https://crrev.com/d12dbab46618801b38430b614276002c178abf72
Cr-Commit-Position: refs/heads/master@{#33911}

[Igor Sheludko, 2016-02-11 14:12:25]

Description was changed from

==========
[es6] More efficient way of marking AST nodes in tail positions.

Instead of doing a full function body traversal we collect return expressions
and mark them after function parsing.

BUG=v8:4698
LOG=N
==========

to

==========
[es6] More efficient way of marking AST call expressions in tail positions.

Instead of doing a full function body traversal we collect return expressions
and mark them after function parsing.

BUG=v8:4698
LOG=N
==========

[Igor Sheludko, 2016-02-11 14:16:20]

Patchset #1 (id:1) has been deleted

[Igor Sheludko, 2016-02-11 14:19:03]

Description was changed from

==========
[es6] More efficient way of marking AST call expressions in tail positions.

Instead of doing a full function body traversal we collect return expressions
and mark them after function parsing.

BUG=v8:4698
LOG=N
==========

to

==========
[es6] More efficient way of marking AST call expressions in tail positions.

Instead of doing a full function body traversal we collect return expressions
and mark them after function parsing.
This CL partially reverts 3c71bd18.

BUG=v8:4698
LOG=N
==========

[Igor Sheludko, 2016-02-11 14:19:33]

Description was changed from

==========
[es6] More efficient way of marking AST call expressions in tail positions.

Instead of doing a full function body traversal we collect return expressions
and mark them after function parsing.
This CL partially reverts 3c71bd18.

BUG=v8:4698
LOG=N
==========

to

==========
[es6] More efficient way of marking AST call expressions in tail positions.

Instead of doing a full function body traversal we collect return expressions
and mark them after function parsing.
This CL partially reverts 3c71bd18 (r33761).

BUG=v8:4698
LOG=N
==========

[Igor Sheludko, 2016-02-11 14:20:31]

Description was changed from

==========
[es6] More efficient way of marking AST call expressions in tail positions.

Instead of doing a full function body traversal we collect return expressions
and mark them after function parsing.
This CL partially reverts 3c71bd18 (r33761).

BUG=v8:4698
LOG=N
==========

to

==========
[es6] More efficient way of marking AST call expressions in tail positions.

Instead of doing a full function body traversal we collect return expressions
and mark them after function parsing.
This CL partially reverts commit 3c71bd184638deab45748e3a0776e4470ddef879
(r33761).

BUG=v8:4698
LOG=N
==========

[Igor Sheludko, 2016-02-11 14:27:35]

Patchset #1 (id:20001) has been deleted

[Igor Sheludko, 2016-02-11 14:30:58]

Patchset #1 (id:40001) has been deleted

[Igor Sheludko, 2016-02-11 14:31:13]

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

[commit-bot: I haz the power, 2016-02-11 14:31:23]

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

[Igor Sheludko, 2016-02-11 14:31:31]

ishell@chromium.org changed reviewers:
+ rossberg@chromium.org

[Igor Sheludko, 2016-02-11 14:31:32]

PTAL

[commit-bot: I haz the power, 2016-02-11 14:54:30]

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

[commit-bot: I haz the power, 2016-02-11 14:54:31]

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

[Igor Sheludko, 2016-02-11 15:21:18]

Description was changed from

==========
[es6] More efficient way of marking AST call expressions in tail positions.

Instead of doing a full function body traversal we collect return expressions
and mark them after function parsing.
This CL partially reverts commit 3c71bd184638deab45748e3a0776e4470ddef879
(r33761).

BUG=v8:4698
LOG=N
==========

to

==========
[es6] More efficient way of marking AST call expressions in tail positions.

Instead of doing a full function body traversal we collect return expressions
and mark them after function parsing.

And since we rewrite do-expressions so that the result is explicitly assigned to
a result variable the statements marking will never hit so I removed it from the
AST.

BUG=v8:4698
LOG=N
==========

[Igor Sheludko, 2016-02-11 15:27:24]

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

[commit-bot: I haz the power, 2016-02-11 15:27:27]

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

[commit-bot: I haz the power, 2016-02-11 16:23:56]

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

[commit-bot: I haz the power, 2016-02-11 16:23:57]

Dry run: This issue passed the CQ dry run.

[rossberg, 2016-02-11 16:44:52]

lgtm

[Igor Sheludko, 2016-02-11 17:38:24]

The CQ bit was checked by ishell@chromium.org

[commit-bot: I haz the power, 2016-02-11 17:38:32]

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

[commit-bot: I haz the power, 2016-02-11 17:39:53]

Description was changed from

==========
[es6] More efficient way of marking AST call expressions in tail positions.

Instead of doing a full function body traversal we collect return expressions
and mark them after function parsing.

And since we rewrite do-expressions so that the result is explicitly assigned to
a result variable the statements marking will never hit so I removed it from the
AST.

BUG=v8:4698
LOG=N
==========

to

==========
[es6] More efficient way of marking AST call expressions in tail positions.

Instead of doing a full function body traversal we collect return expressions
and mark them after function parsing.

And since we rewrite do-expressions so that the result is explicitly assigned to
a result variable the statements marking will never hit so I removed it from the
AST.

BUG=v8:4698
LOG=N
==========

[commit-bot: I haz the power, 2016-02-11 17:39:53]

Committed patchset #2 (id:80001)

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

Description was changed from

==========
[es6] More efficient way of marking AST call expressions in tail positions.

Instead of doing a full function body traversal we collect return expressions
and mark them after function parsing.

And since we rewrite do-expressions so that the result is explicitly assigned to
a result variable the statements marking will never hit so I removed it from the
AST.

BUG=v8:4698
LOG=N
==========

to

==========
[es6] More efficient way of marking AST call expressions in tail positions.

Instead of doing a full function body traversal we collect return expressions
and mark them after function parsing.

And since we rewrite do-expressions so that the result is explicitly assigned to
a result variable the statements marking will never hit so I removed it from the
AST.

BUG=v8:4698
LOG=N

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

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

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