[wasm] Refactor WASM run tests to allow them to run in the interpreter too.

[wasm] Refactor WASM run tests to allow them to run in the interpreter too.

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

Committed: https://crrev.com/3fef34e02388e07d46067c516320f1ff12304c8e
Cr-Commit-Position: refs/heads/master@{#36332}

[Michael Starzinger, 2016-05-18 15:33:44]

LGTM.

[ahaas, 2016-05-18 15:43:26]

lgtm

https://codereview.chromium.org/1990923002/diff/1/test/cctest/wasm/wasm-run-u...
File test/cctest/wasm/wasm-run-utils.h (right):

https://codereview.chromium.org/1990923002/diff/1/test/cctest/wasm/wasm-run-u...
test/cctest/wasm/wasm-run-utils.h:667: TEST(RunWasmCompiled_##name) {
RunWasm_##name(); } \
Could we use shorter test names here? Maybe RunWasm_##name and
InterpretWasm_##name?

[titzer, 2016-05-18 15:53:42]

On 2016/05/18 15:43:26, ahaas wrote:
> lgtm
> 
>
https://codereview.chromium.org/1990923002/diff/1/test/cctest/wasm/wasm-run-u...
> File test/cctest/wasm/wasm-run-utils.h (right):
> 
>
https://codereview.chromium.org/1990923002/diff/1/test/cctest/wasm/wasm-run-u...
> test/cctest/wasm/wasm-run-utils.h:667: TEST(RunWasmCompiled_##name) {
> RunWasm_##name(); } \
> Could we use shorter test names here? Maybe RunWasm_##name and
> InterpretWasm_##name?

Eh, they both run the code, so might as well be explicit about it.

[titzer, 2016-05-18 15:53:47]

The CQ bit was checked by titzer@chromium.org

[commit-bot: I haz the power, 2016-05-18 15:53:55]

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

[commit-bot: I haz the power, 2016-05-18 15:56:16]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-05-18 15:57:12]

Description was changed from

==========
[wasm] Refactor WASM run tests to allow them to run in the interpreter too.

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

to

==========
[wasm] Refactor WASM run tests to allow them to run in the interpreter too.

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

Committed: https://crrev.com/3fef34e02388e07d46067c516320f1ff12304c8e
Cr-Commit-Position: refs/heads/master@{#36332}
==========

[commit-bot: I haz the power, 2016-05-18 15:57:13]

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