Get rid of the different kinds of yield in the AST & full-codegen.

Get rid of the different kinds of yield in the AST & full-codegen.

Now there is just one kind, corresponding to what was called "initial" before.
Replacement for "suspend": when the parser sees a yield in JS code, it
will turn it into a Yield node but wrap its argument in an iterator result
object.  Replacement for "final": the parser simply inserts a return statement
instead.

R=littledan@chromium.org, mstarzinger@chromium.org
BUG=

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

[neis, 2016-03-01 15:00:43]

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

[commit-bot: I haz the power, 2016-03-01 15:00:53]

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

[Michael Starzinger, 2016-03-01 15:03:25]

LGTM on the back-ends, didn't look at the parser all too much.

https://codereview.chromium.org/1751613004/diff/1/src/parsing/parser.cc
File src/parsing/parser.cc (right):

https://codereview.chromium.org/1751613004/diff/1/src/parsing/parser.cc#newco...
src/parsing/parser.cc:505: return
factory->NewCallRuntime(Runtime::kCreateIterResultObject, args, pos);
nit: I think we want kInlineCreateIterResultObject here, right?

[commit-bot: I haz the power, 2016-03-01 15:24:32]

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

[commit-bot: I haz the power, 2016-03-01 15:24:34]

Dry run: This issue passed the CQ dry run.

[neis, 2016-03-01 15:26:42]

https://codereview.chromium.org/1751613004/diff/1/src/parsing/parser.cc
File src/parsing/parser.cc (right):

https://codereview.chromium.org/1751613004/diff/1/src/parsing/parser.cc#newco...
src/parsing/parser.cc:505: return
factory->NewCallRuntime(Runtime::kCreateIterResultObject, args, pos);
On 2016/03/01 15:03:25, Michael Starzinger wrote:
> nit: I think we want kInlineCreateIterResultObject here, right?

Done.

[neis, 2016-03-04 09:26:10]

Dan, can you please have a look at the parser changes?

[Dan Ehrenberg, 2016-03-04 17:21:49]

Parser lgtm

[neis, 2016-03-06 08:26:29]

The CQ bit was checked by neis@chromium.org

[neis, 2016-03-06 08:26:30]

The patchset sent to the CQ was uploaded after l-g-t-m from
mstarzinger@chromium.org
Link to the patchset: https://codereview.chromium.org/1751613004/#ps40001
(title: "Do an inline call.")

[commit-bot: I haz the power, 2016-03-06 08:26:41]

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

[commit-bot: I haz the power, 2016-03-06 09:19:27]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-03-06 09:20:21]

Description was changed from

==========
Get rid of the different kinds of yield in the AST & full-codegen.

Now there is just one kind, corresponding to what was called "initial" before.
Replacement for "suspend": when the parser sees a yield in JS code, it
will turn it into a Yield node but wrap its argument in an iterator result
object.  Replacement for "final": the parser simply inserts a return statement
instead.

R=littledan@chromium.org, mstarzinger@chromium.org
BUG=
==========

to

==========
Get rid of the different kinds of yield in the AST & full-codegen.

Now there is just one kind, corresponding to what was called "initial" before.
Replacement for "suspend": when the parser sees a yield in JS code, it
will turn it into a Yield node but wrap its argument in an iterator result
object.  Replacement for "final": the parser simply inserts a return statement
instead.

R=littledan@chromium.org, mstarzinger@chromium.org
BUG=

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

[commit-bot: I haz the power, 2016-03-06 09:20:22]

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