[compiler] Sidestep the interpreter for generator functions.

[compiler] Sidestep the interpreter for generator functions.

This ensures the interpreter is not tasked with compiling generator
functions. It currently does not support suspending activations at
yielding points, but we still want to be able to activate it for the
rest of JavaScript in the meantime.

R=rmcilroy@chromium.org
BUG=v8:4681
LOG=n

Committed: https://crrev.com/899105c0bc92c9785284cfeb5bcbfec9a370995b
Cr-Commit-Position: refs/heads/master@{#34672}

[Michael Starzinger, 2016-03-10 09:28:14]

Description was changed from

==========
[compiler] Sidestep the interpreter for generator function.

This ensures the interpreter is not tasked with compiling generator
functions. It currently does not support suspending activations at
yielding points, but we still want to be able to activate it for the
rest of JavaScript in the meantime.

R=rmcilroy@chromium.org
BUG=v8:4681
LOG=n
==========

to

==========
[compiler] Sidestep the interpreter for generator functions.

This ensures the interpreter is not tasked with compiling generator
functions. It currently does not support suspending activations at
yielding points, but we still want to be able to activate it for the
rest of JavaScript in the meantime.

R=rmcilroy@chromium.org
BUG=v8:4681
LOG=n
==========

[rmcilroy, 2016-03-10 10:15:24]

Could you also update test262.status as well please? I've just cced you on a
thread where we found out that test262 for Ignition doesn't seem to be running
any of the tests on the bots, but it should work locally.

LGTM on code changes.

https://codereview.chromium.org/1782013002/diff/20001/test/mjsunit/mjsunit.st...
File test/mjsunit/mjsunit.status (right):

https://codereview.chromium.org/1782013002/diff/20001/test/mjsunit/mjsunit.st...
test/mjsunit/mjsunit.status:827: 'messages': [FAIL],
Any idea why the rest of these still fail?

[Michael Starzinger, 2016-03-10 10:42:40]

Yes, will adapt test262 expectations as well.

https://codereview.chromium.org/1782013002/diff/20001/test/mjsunit/mjsunit.st...
File test/mjsunit/mjsunit.status (right):

https://codereview.chromium.org/1782013002/diff/20001/test/mjsunit/mjsunit.st...
test/mjsunit/mjsunit.status:827: 'messages': [FAIL],
On 2016/03/10 10:15:24, rmcilroy wrote:
> Any idea why the rest of these still fail?

Acknowledged. Yes, I know what this is. It is the generator builtin methods that
resume an activation. I am working on fixing this. Stay tuned! :)

[Michael Starzinger, 2016-03-10 13:08:39]

The CQ bit was checked by mstarzinger@chromium.org

[Michael Starzinger, 2016-03-10 13:08:40]

The patchset sent to the CQ was uploaded after l-g-t-m from
rmcilroy@chromium.org
Link to the patchset: https://codereview.chromium.org/1782013002/#ps80001
(title: "Rebased.")

[commit-bot: I haz the power, 2016-03-10 13:08:43]

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

[commit-bot: I haz the power, 2016-03-10 13:20:41]

Description was changed from

==========
[compiler] Sidestep the interpreter for generator functions.

This ensures the interpreter is not tasked with compiling generator
functions. It currently does not support suspending activations at
yielding points, but we still want to be able to activate it for the
rest of JavaScript in the meantime.

R=rmcilroy@chromium.org
BUG=v8:4681
LOG=n
==========

to

==========
[compiler] Sidestep the interpreter for generator functions.

This ensures the interpreter is not tasked with compiling generator
functions. It currently does not support suspending activations at
yielding points, but we still want to be able to activate it for the
rest of JavaScript in the meantime.

R=rmcilroy@chromium.org
BUG=v8:4681
LOG=n
==========

[commit-bot: I haz the power, 2016-03-10 13:20:41]

Committed patchset #5 (id:80001)

[commit-bot: I haz the power, 2016-03-10 13:21:59]

Description was changed from

==========
[compiler] Sidestep the interpreter for generator functions.

This ensures the interpreter is not tasked with compiling generator
functions. It currently does not support suspending activations at
yielding points, but we still want to be able to activate it for the
rest of JavaScript in the meantime.

R=rmcilroy@chromium.org
BUG=v8:4681
LOG=n
==========

to

==========
[compiler] Sidestep the interpreter for generator functions.

This ensures the interpreter is not tasked with compiling generator
functions. It currently does not support suspending activations at
yielding points, but we still want to be able to activate it for the
rest of JavaScript in the meantime.

R=rmcilroy@chromium.org
BUG=v8:4681
LOG=n

Committed: https://crrev.com/899105c0bc92c9785284cfeb5bcbfec9a370995b
Cr-Commit-Position: refs/heads/master@{#34672}
==========

[commit-bot: I haz the power, 2016-03-10 13:22:00]

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