Support blocks in asm->wasm global section.

Support blocks in asm->wasm global section.

The parser can fold together comma separated declarations
into a single block. The AsmWasmBuilder needs to support
this case in order to handle asm.js modules that use this form.

BUG= https://code.google.com/p/v8/issues/detail?id=4203
TEST=mjsunit/asm-wasm
R=aseemgarg@chromium.org,titzer@chromium.org
LOG=N

Committed: https://crrev.com/6357ad847bf6a6f6d9cbbc9fb1077623641283ff
Cr-Commit-Position: refs/heads/master@{#34048}

[bradn, 2016-02-16 16:49:23]

The CQ bit was checked by bradnelson@google.com to run a CQ dry run

[commit-bot: I haz the power, 2016-02-16 16:49:36]

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

[titzer, 2016-02-16 16:53:01]

On 2016/02/16 16:49:36, commit-bot: I haz the power wrote:
> Dry run: CQ is trying da patch. Follow status at
>  https://chromium-cq-status.appspot.com/patch-status/1697423003/1
> View timeline at
>  https://chromium-cq-status.appspot.com/patch-timeline/1697423003/1

lgtm

[commit-bot: I haz the power, 2016-02-16 17:13:03]

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

[commit-bot: I haz the power, 2016-02-16 17:13:03]

Dry run: This issue passed the CQ dry run.

[bradnelson, 2016-02-16 18:32:51]

The CQ bit was checked by bradnelson@chromium.org

[commit-bot: I haz the power, 2016-02-16 18:33:00]

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

[commit-bot: I haz the power, 2016-02-16 18:35:13]

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

[commit-bot: I haz the power, 2016-02-16 18:35:15]

Failed to apply patch for test/mjsunit/wasm/asm-wasm.js:
While running git apply --index -3 -p1;
  error: patch failed: test/mjsunit/wasm/asm-wasm.js:1219
  Falling back to three-way merge...
  Applied patch to 'test/mjsunit/wasm/asm-wasm.js' with conflicts.
  U test/mjsunit/wasm/asm-wasm.js

Patch:       test/mjsunit/wasm/asm-wasm.js
Index: test/mjsunit/wasm/asm-wasm.js
diff --git a/test/mjsunit/wasm/asm-wasm.js b/test/mjsunit/wasm/asm-wasm.js
index
100da9d065bfa3ea7446ef3d25ea0b293a2591cc..b81618152743854461c204adc5735d2fa308a1a7
100644
--- a/test/mjsunit/wasm/asm-wasm.js
+++ b/test/mjsunit/wasm/asm-wasm.js
@@ -1219,3 +1219,23 @@ TestForeignVariables();
   assertEquals(42, m.iload(4));
   assertEquals(77, m.iload(8));
 })();
+
+
+(function TestGlobalBlock() {
+  function Module(stdlib, foreign, buffer) {
+    "use asm";
+
+    var x = foreign.x | 0, y = foreign.y | 0;
+
+    function test() {
+      return (x + y) | 0;
+    }
+
+    return {test: test};
+  }
+
+  var m = _WASMEXP_.instantiateModuleFromAsm(
+      Module.toString(), { x: 4, y: 11 });
+  m.__init__();
+  assertEquals(15, m.test());
+})();

[bradnelson, 2016-02-16 19:02:52]

The CQ bit was checked by bradnelson@chromium.org

[bradnelson, 2016-02-16 19:02:52]

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/1697423003/#ps20001
(title: "merge")

[commit-bot: I haz the power, 2016-02-16 19:03:02]

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

[commit-bot: I haz the power, 2016-02-16 19:33:34]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-02-16 19:33:57]

Description was changed from

==========
Support blocks in asm->wasm global section.

The parser can fold together comma separated declarations
into a single block. The AsmWasmBuilder needs to support
this case in order to handle asm.js modules that use this form.

BUG= https://code.google.com/p/v8/issues/detail?id=4203
TEST=mjsunit/asm-wasm
R=aseemgarg@chromium.org,titzer@chromium.org
LOG=N
==========

to

==========
Support blocks in asm->wasm global section.

The parser can fold together comma separated declarations
into a single block. The AsmWasmBuilder needs to support
this case in order to handle asm.js modules that use this form.

BUG= https://code.google.com/p/v8/issues/detail?id=4203
TEST=mjsunit/asm-wasm
R=aseemgarg@chromium.org,titzer@chromium.org
LOG=N

Committed: https://crrev.com/6357ad847bf6a6f6d9cbbc9fb1077623641283ff
Cr-Commit-Position: refs/heads/master@{#34048}
==========

[commit-bot: I haz the power, 2016-02-16 19:33:58]

Patchset 2 (id:??) landed as
https://crrev.com/6357ad847bf6a6f6d9cbbc9fb1077623641283ff
Cr-Commit-Position: refs/heads/master@{#34048}