[wasm] [asm.js] Track direct function indices and fixup.

[wasm] [asm.js] Track direct function indices and fixup.

Recent changes to the wasm format prevent you from knowing
the function index of a direct call until you know how many
imports you have (the index spaces have been merged).

Asm.js validation can't know this until all callsites have been
visited if you want to avoid materializing the entire AST / parsing twice.
Instead, keep a list of fixups to perform for all direct callsites,
then update these with the imports count added when emitting the module.

BUG=v8:4203
R=aseemgarg@chromium.org,titzer@chromium.org

Committed: https://crrev.com/cfed56c2c539fc2a103423169212999b1e7143c5
Cr-Commit-Position: refs/heads/master@{#39937}

[bradnelson, 2016-09-30 23:02:16]

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

[commit-bot: I haz the power, 2016-09-30 23:02:21]

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, 2016-09-30 23:10:22]

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

[commit-bot: I haz the power, 2016-09-30 23:10:23]

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...)

[bradnelson, 2016-10-03 17:30:53]

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

[bradn, 2016-10-03 17:30:59]

bradnelson@google.com changed reviewers:
+ bradnelson@google.com

[bradn, 2016-10-03 17:30:59]

Ping?

[commit-bot: I haz the power, 2016-10-03 17:31:04]

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, 2016-10-03 17:54:57]

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

[commit-bot: I haz the power, 2016-10-03 17:54:58]

Dry run: This issue passed the CQ dry run.

[aseemgarg, 2016-10-03 19:38:59]

lgtm

[bradnelson, 2016-10-03 20:59:18]

The CQ bit was checked by bradnelson@chromium.org

[commit-bot: I haz the power, 2016-10-03 20:59:24]

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, 2016-10-03 21:04:33]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-10-03 21:04:56]

Description was changed from

==========
[wasm] [asm.js] Track direct function indices and fixup.

Recent changes to the wasm format prevent you from knowing
the function index of a direct call until you know how many
imports you have (the index spaces have been merged).

Asm.js validation can't know this until all callsites have been
visited if you want to avoid materializing the entire AST / parsing twice.
Instead, keep a list of fixups to perform for all direct callsites,
then update these with the imports count added when emitting the module.

BUG=v8:4203
R=aseemgarg@chromium.org,titzer@chromium.org
==========

to

==========
[wasm] [asm.js] Track direct function indices and fixup.

Recent changes to the wasm format prevent you from knowing
the function index of a direct call until you know how many
imports you have (the index spaces have been merged).

Asm.js validation can't know this until all callsites have been
visited if you want to avoid materializing the entire AST / parsing twice.
Instead, keep a list of fixups to perform for all direct callsites,
then update these with the imports count added when emitting the module.

BUG=v8:4203
R=aseemgarg@chromium.org,titzer@chromium.org

Committed: https://crrev.com/cfed56c2c539fc2a103423169212999b1e7143c5
Cr-Commit-Position: refs/heads/master@{#39937}
==========

[commit-bot: I haz the power, 2016-10-03 21:04:57]

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