[interpreter] Adds interpreter cctests.

[interpreter] Adds interpreter cctests.

BUG=v8:4280
LOG=N

Committed: https://crrev.com/6447b78f8e95520e063099d90625bc8b33384532
Cr-Commit-Position: refs/heads/master@{#30087}

[rmcilroy, 2015-07-30 08:09:04]

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

[rmcilroy, 2015-07-30 08:09:04]

Orion: This is the CL I mentioned which would be really nice to have a
BytecodeEmitter for :).

[oth, 2015-07-30 09:49:01]

Thanks, will use as template for bytecode generation tests.

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

https://codereview.chromium.org/1269683002/diff/1/test/cctest/interpreter/tes...
test/cctest/interpreter/test-interpreter.cc:1: // Copyright 2014 the V8 project
authors. All rights reserved.
s/2014/2015/

[rmcilroy, 2015-07-30 11:02:17]

> Thanks, will use as template for bytecode generation tests.

Great, you might also want to look at function-tester.h since you will
presumably just be compiling straight from JS source (with the interpreter flag
set appropriately to ensure it goes through the interpreter).

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

https://codereview.chromium.org/1269683002/diff/1/test/cctest/interpreter/tes...
test/cctest/interpreter/test-interpreter.cc:1: // Copyright 2014 the V8 project
authors. All rights reserved.
On 2015/07/30 09:49:01, oth wrote:
> s/2014/2015/

Done, thanks.

[rmcilroy, 2015-07-31 12:05:58]

Patchset #3 (id:40001) has been deleted

[rmcilroy, 2015-07-31 12:06:05]

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

[commit-bot: I haz the power, 2015-07-31 12:06:11]

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

[rmcilroy, 2015-07-31 12:06:22]

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

[rmcilroy, 2015-07-31 12:06:22]

Ben, could you take a look please, thanks.

[rmcilroy, 2015-07-31 12:06:26]

Ben, could you take a look please, thanks.

[titzer, 2015-07-31 12:09:12]

https://codereview.chromium.org/1269683002/diff/60001/test/cctest/interpreter...
File test/cctest/interpreter/test-interpreter.cc (right):

https://codereview.chromium.org/1269683002/diff/60001/test/cctest/interpreter...
test/cctest/interpreter/test-interpreter.cc:31: MaybeHandle<Object> Call() {
Can we factor the Call() out of the tester?

This is one of the design mistakes I think we made with the way the other
integration tests work. The tester should produce a Callable thingy, which has a
Call() or an operator()

[commit-bot: I haz the power, 2015-07-31 12:10:06]

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

[commit-bot: I haz the power, 2015-07-31 12:10:06]

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

[rmcilroy, 2015-07-31 17:52:59]

PTAL, thanks.

https://codereview.chromium.org/1269683002/diff/60001/test/cctest/interpreter...
File test/cctest/interpreter/test-interpreter.cc (right):

https://codereview.chromium.org/1269683002/diff/60001/test/cctest/interpreter...
test/cctest/interpreter/test-interpreter.cc:31: MaybeHandle<Object> Call() {
On 2015/07/31 12:09:12, titzer wrote:
> Can we factor the Call() out of the tester?
> 
> This is one of the design mistakes I think we made with the way the other
> integration tests work. The tester should produce a Callable thingy, which has
a
> Call() or an operator()

Done.

[rmcilroy, 2015-07-31 17:53:35]

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

[commit-bot: I haz the power, 2015-07-31 17:53:50]

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

[commit-bot: I haz the power, 2015-07-31 18:07:00]

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

[commit-bot: I haz the power, 2015-07-31 18:07:01]

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

[rmcilroy, 2015-07-31 21:02:37]

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

[commit-bot: I haz the power, 2015-07-31 21:02:49]

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

[commit-bot: I haz the power, 2015-07-31 21:26:38]

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

[commit-bot: I haz the power, 2015-07-31 21:26:38]

Dry run: This issue passed the CQ dry run.

[titzer, 2015-08-03 09:46:30]

lgtm

[oth, 2015-08-03 09:52:54]

The CQ bit was checked by oth@chromium.org

[commit-bot: I haz the power, 2015-08-03 09:53:02]

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

[commit-bot: I haz the power, 2015-08-03 09:53:02]

This CL has an open dependency (Issue 1266713004 Patch 60001). Please resolve
the dependency and try again.

[rmcilroy, 2015-08-10 10:55:01]

The CQ bit was checked by rmcilroy@chromium.org

[rmcilroy, 2015-08-10 10:55:02]

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

[commit-bot: I haz the power, 2015-08-10 10:55:07]

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

[commit-bot: I haz the power, 2015-08-10 11:20:27]

Committed patchset #6 (id:120001)

[commit-bot: I haz the power, 2015-08-10 11:20:37]

Patchset 6 (id:??) landed as
https://crrev.com/6447b78f8e95520e063099d90625bc8b33384532
Cr-Commit-Position: refs/heads/master@{#30087}