[wasm] Use pimpl idiom for interpreter thread implementation

[wasm] Use pimpl idiom for interpreter thread implementation

We used dynamic dispatch before, which has considerable runtime
overhead. Pimpl (a.k.a. opaque pointer) has the same memory overhead
of one additional pointer, but less runtime overhead.

R=titzer@chromium.org

Review-Url: https://codereview.chromium.org/2639013002
Cr-Commit-Position: refs/heads/master@{#42442}
Committed: https://chromium.googlesource.com/v8/v8/+/7634b0eb13e945191c04e63af86bf9fdc1f6aa4c

[Clemens Hammacher, 2017-01-17 19:55:15]

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

[Clemens Hammacher, 2017-01-17 20:01:44]

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

[commit-bot: I haz the power, 2017-01-17 20:01:50]

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 20:07:11]

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

[commit-bot: I haz the power, 2017-01-17 20:07:11]

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

[Clemens Hammacher, 2017-01-18 09:39:17]

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

[Clemens Hammacher, 2017-01-18 09:39:19]

Description was changed from

==========
[wasm] Use pimpl idiom for interpreter thread implementation

We used dynamic dispatch before, which has considerable runtime
overhead. Pimpl (a.k.a. opaque pointer) has the same memory overhead
of one additional pointer, but less runtime overhead.

The change in zone-allocator.h is needed to support emplace_back in the
ZoneVector for any constructor which is not the copy-constructor.

R=titzer@chromium.org
==========

to

==========
[wasm] Use pimpl idiom for interpreter thread implementation

We used dynamic dispatch before, which has considerable runtime
overhead. Pimpl (a.k.a. opaque pointer) has the same memory overhead
of one additional pointer, but less runtime overhead.

R=titzer@chromium.org
==========

[commit-bot: I haz the power, 2017-01-18 09:39:29]

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

[titzer, 2017-01-18 09:53:21]

lgtm

[commit-bot: I haz the power, 2017-01-18 10:05:45]

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

[commit-bot: I haz the power, 2017-01-18 10:05:45]

Dry run: This issue passed the CQ dry run.

[Clemens Hammacher, 2017-01-18 10:21:35]

The CQ bit was checked by clemensh@chromium.org

[commit-bot: I haz the power, 2017-01-18 10:21:45]

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-18 10:23:21]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1484734895773280,
"parent_rev": "c5caaf8921ac4a0c0a9ae16a1a0d1962866d2b90", "commit_rev":
"7634b0eb13e945191c04e63af86bf9fdc1f6aa4c"}

[commit-bot: I haz the power, 2017-01-18 10:23:25]

Description was changed from

==========
[wasm] Use pimpl idiom for interpreter thread implementation

We used dynamic dispatch before, which has considerable runtime
overhead. Pimpl (a.k.a. opaque pointer) has the same memory overhead
of one additional pointer, but less runtime overhead.

R=titzer@chromium.org
==========

to

==========
[wasm] Use pimpl idiom for interpreter thread implementation

We used dynamic dispatch before, which has considerable runtime
overhead. Pimpl (a.k.a. opaque pointer) has the same memory overhead
of one additional pointer, but less runtime overhead.

R=titzer@chromium.org

Review-Url: https://codereview.chromium.org/2639013002
Cr-Commit-Position: refs/heads/master@{#42442}
Committed:
https://chromium.googlesource.com/v8/v8/+/7634b0eb13e945191c04e63af86bf9fdc1f...
==========

[commit-bot: I haz the power, 2017-01-18 10:23:26]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/v8/v8/+/7634b0eb13e945191c04e63af86bf9fdc1f...