[wasm] Skip whole function linking phase

[wasm] Skip whole function linking phase

As functions are relinked at instantiation anyway, we can just
completely skip the LinkFunctions pass at wasm compile time.
This way, we also don't need to create one placeholder per wasm
function, but we just use one placeholder everywhere. The function
index which was stored in the placeholder before is now retrieved from
the wire bytes anyway.

R=titzer@chromium.org

Review-Url: https://codereview.chromium.org/2624383003
Cr-Commit-Position: refs/heads/master@{#42285}
Committed: https://chromium.googlesource.com/v8/v8/+/723aeeaba2d672657675306004b9e91fdc396ee6

[Clemens Hammacher, 2017-01-12 14:36:46]

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

[commit-bot: I haz the power, 2017-01-12 14:36: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-12 14:43:39]

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

[commit-bot: I haz the power, 2017-01-12 14:43:40]

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

[Clemens Hammacher, 2017-01-12 15:11:45]

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

[commit-bot: I haz the power, 2017-01-12 15:11:52]

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-12 15:17:02]

Yes, this is awesome! Bye bye placeholders!
LGTM

[commit-bot: I haz the power, 2017-01-12 15:55:35]

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

[commit-bot: I haz the power, 2017-01-12 15:55:36]

Dry run: This issue passed the CQ dry run.

[Clemens Hammacher, 2017-01-12 19:27:53]

The CQ bit was checked by clemensh@chromium.org

[commit-bot: I haz the power, 2017-01-12 19:27:56]

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-12 19:29:54]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1484249273080100,
"parent_rev": "a0e3a8ee89c940e66ccafa6d4e9ee664d91e9a3b", "commit_rev":
"723aeeaba2d672657675306004b9e91fdc396ee6"}

[commit-bot: I haz the power, 2017-01-12 19:29:59]

Description was changed from

==========
[wasm] Skip whole function linking phase

As functions are relinked at instantiation anyway, we can just
completely skip the LinkFunctions pass at wasm compile time.
This way, we also don't need to create one placeholder per wasm
function, but we just use one placeholder everywhere. The function
index which was stored in the placeholder before is now retrieved from
the wire bytes anyway.

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

to

==========
[wasm] Skip whole function linking phase

As functions are relinked at instantiation anyway, we can just
completely skip the LinkFunctions pass at wasm compile time.
This way, we also don't need to create one placeholder per wasm
function, but we just use one placeholder everywhere. The function
index which was stored in the placeholder before is now retrieved from
the wire bytes anyway.

R=titzer@chromium.org

Review-Url: https://codereview.chromium.org/2624383003
Cr-Commit-Position: refs/heads/master@{#42285}
Committed:
https://chromium.googlesource.com/v8/v8/+/723aeeaba2d672657675306004b9e91fdc3...
==========

[commit-bot: I haz the power, 2017-01-12 19:30:00]

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