[Interpreter] Add bytecode generator expectations for super calls.

[Interpreter] Add bytecode generator expectations for super calls.

BUG=v8:4682
LOG=N

Committed: https://crrev.com/18109232c1cddfa8b076dc8901fc02d983a1f0a7
Cr-Commit-Position: refs/heads/master@{#34819}

[rmcilroy, 2016-03-15 14:07:34]

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

[commit-bot: I haz the power, 2016-03-15 14:07:40]

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

[rmcilroy, 2016-03-15 14:07:43]

rmcilroy@chromium.org changed reviewers:
+ oth@chromium.org

[rmcilroy, 2016-03-15 14:07:44]

PTAL, thanks.

[commit-bot: I haz the power, 2016-03-15 14:27:32]

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

[commit-bot: I haz the power, 2016-03-15 14:27:32]

Dry run: This issue passed the CQ dry run.

[oth, 2016-03-15 14:54:09]

lgtm with minor comments. A few more easy cases would be good - mjsunit has so
many many variations that don't need reproing in entirety here, but cover
execution well.

https://codereview.chromium.org/1805503003/diff/1/test/cctest/interpreter/tes...
File test/cctest/interpreter/test-bytecode-generator.cc (right):

https://codereview.chromium.org/1805503003/diff/1/test/cctest/interpreter/tes...
test/cctest/interpreter/test-bytecode-generator.cc:2140: const char* snippets[]
= {
Can you add a few more here - InterpreterClassAndSuperClass has calls to parent
constructor with and without arguments, setting properties on super. These would
be good to have here too.

https://codereview.chromium.org/1805503003/diff/1/test/cctest/interpreter/tes...
File test/cctest/interpreter/test-interpreter.cc (right):

https://codereview.chromium.org/1805503003/diff/1/test/cctest/interpreter/tes...
test/cctest/interpreter/test-interpreter.cc:3928: InterpreterTester
tester(handles.main_isolate(), source.c_str());
"*"

[rmcilroy, 2016-03-16 14:16:41]

https://codereview.chromium.org/1805503003/diff/1/test/cctest/interpreter/tes...
File test/cctest/interpreter/test-bytecode-generator.cc (right):

https://codereview.chromium.org/1805503003/diff/1/test/cctest/interpreter/tes...
test/cctest/interpreter/test-bytecode-generator.cc:2140: const char* snippets[]
= {
On 2016/03/15 14:54:09, oth wrote:
> Can you add a few more here - InterpreterClassAndSuperClass has calls to
parent
> constructor with and without arguments, setting properties on super. These
would
> be good to have here too.

Done.

https://codereview.chromium.org/1805503003/diff/1/test/cctest/interpreter/tes...
File test/cctest/interpreter/test-interpreter.cc (right):

https://codereview.chromium.org/1805503003/diff/1/test/cctest/interpreter/tes...
test/cctest/interpreter/test-interpreter.cc:3928: InterpreterTester
tester(handles.main_isolate(), source.c_str());
On 2016/03/15 14:54:09, oth wrote:
> "*"

Done.

[rmcilroy, 2016-03-16 14:17:00]

The CQ bit was checked by rmcilroy@chromium.org

[rmcilroy, 2016-03-16 14:17:00]

The patchset sent to the CQ was uploaded after l-g-t-m from oth@chromium.org
Link to the patchset: https://codereview.chromium.org/1805503003/#ps20001
(title: "Add additional test cases.")

[commit-bot: I haz the power, 2016-03-16 14:17:06]

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

[oth, 2016-03-16 14:25:12]

Cool, lgtm, thanks.

[commit-bot: I haz the power, 2016-03-16 14:41:13]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-03-16 14:43:04]

Description was changed from

==========
[Interpreter] Add bytecode generator expectations for super calls.

BUG=v8:4682
LOG=N
==========

to

==========
[Interpreter] Add bytecode generator expectations for super calls.

BUG=v8:4682
LOG=N

Committed: https://crrev.com/18109232c1cddfa8b076dc8901fc02d983a1f0a7
Cr-Commit-Position: refs/heads/master@{#34819}
==========

[commit-bot: I haz the power, 2016-03-16 14:43:05]

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