[generators] Always call function with closure context when resuming.

[generators] Always call function with closure context when resuming.

The resume trampolin used to call the generator function with the context of the
last suspension rather than the closure's context.  While that was fine for
Ignition, Turbofan got utterly confused.  With this CL, the resume trampolin
always passes in the closure's context (like in the very first call of the
generator function).  The generator function itself then restores its previously
current context by reading it from the generator object and doing a
PushContext.

BUG=chromium:681171
Review-Url: https://codereview.chromium.org/2639533002
Cr-Commit-Position: refs/heads/master@{#42407}
Committed: https://chromium.googlesource.com/v8/v8/+/c5948b9897fd3d2fe2cb670375eda59ff7aff0c9

[neis, 2017-01-17 11:55:46]

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

[commit-bot: I haz the power, 2017-01-17 11:55:49]

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

[neis, 2017-01-17 12:02:05]

neis@chromium.org changed reviewers:
+ jarin@chromium.org, mstarzinger@chromium.org

[neis, 2017-01-17 12:02:06]

PTAL (other ports to be done)

[Jarin, 2017-01-17 12:13:14]

lgtm

https://codereview.chromium.org/2639533002/diff/1/test/mjsunit/regress/regres...
File test/mjsunit/regress/regress-681171-1.js (right):

https://codereview.chromium.org/2639533002/diff/1/test/mjsunit/regress/regres...
test/mjsunit/regress/regress-681171-1.js:15: "xxx".match();
Please, use some sensible formatting (here and in the other tests).

[Michael Starzinger, 2017-01-17 12:18:35]

LGTM.

[commit-bot: I haz the power, 2017-01-17 12:24:12]

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

[commit-bot: I haz the power, 2017-01-17 12:24:13]

Dry run: This issue passed the CQ dry run.

[neis, 2017-01-17 13:08:11]

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

[commit-bot: I haz the power, 2017-01-17 13: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-17 13:40:34]

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

[commit-bot: I haz the power, 2017-01-17 13:40:35]

Dry run: This issue passed the CQ dry run.

[neis, 2017-01-17 13:42:09]

The CQ bit was checked by neis@chromium.org

[neis, 2017-01-17 13:42:10]

The patchset sent to the CQ was uploaded after l-g-t-m from jarin@chromium.org,
mstarzinger@chromium.org
Link to the patchset: https://codereview.chromium.org/2639533002/#ps40001
(title: "Ports.")

[commit-bot: I haz the power, 2017-01-17 13:42:15]

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

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1484660529963210,
"parent_rev": "8863f9f258623ecc3304f6e3156c829e8960f8a4", "commit_rev":
"c5948b9897fd3d2fe2cb670375eda59ff7aff0c9"}

[commit-bot: I haz the power, 2017-01-17 13:44:15]

Description was changed from

==========
[generators] Always call function with closure context when resuming.

The resume trampolin used to call the generator function with the context of the
last suspension rather than the closure's context.  While that was fine for
Ignition, Turbofan got utterly confused.  With this CL, the resume trampolin
always passes in the closure's context (like in the very first call of the
generator function).  The generator function itself then restores its previously
current context by reading it from the generator object and doing a
PushContext.

BUG=chromium:681171
==========

to

==========
[generators] Always call function with closure context when resuming.

The resume trampolin used to call the generator function with the context of the
last suspension rather than the closure's context.  While that was fine for
Ignition, Turbofan got utterly confused.  With this CL, the resume trampolin
always passes in the closure's context (like in the very first call of the
generator function).  The generator function itself then restores its previously
current context by reading it from the generator object and doing a
PushContext.

BUG=chromium:681171

Review-Url: https://codereview.chromium.org/2639533002
Cr-Commit-Position: refs/heads/master@{#42407}
Committed:
https://chromium.googlesource.com/v8/v8/+/c5948b9897fd3d2fe2cb670375eda59ff7a...
==========

[commit-bot: I haz the power, 2017-01-17 13:44:16]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/v8/v8/+/c5948b9897fd3d2fe2cb670375eda59ff7a...