[wasm] Add support for multiple indirect function tables

test/mjsunit/wasm/wasm-module-builder.js

Index: test/mjsunit/wasm/wasm-module-builder.js
diff --git a/test/mjsunit/wasm/wasm-module-builder.js b/test/mjsunit/wasm/wasm-module-builder.js
index 5c5df798fbc9126c780a50a2a26eb9f37a82f4f6..ea8a026f3f053863a7174430de2cd61f10b94334 100644
--- a/test/mjsunit/wasm/wasm-module-builder.js
+++ b/test/mjsunit/wasm/wasm-module-builder.js
@@ -227,6 +227,10 @@ class WasmModuleBuilder {
       if (debug) print("emitting table @ " + binary.length);
       binary.emit_section(kDeclTable, section => {
         section.emit_varint(wasm.table.length);
+        if (wasm.pad != null) {

[ahaas, 2016/07/25 19:23:45]

Do you intentionally use != instead of !== here?

[ddchen, 2016/07/25 22:17:36]

Nope, good catch!

+          if (debug) print("emitting table padding @ " + binary.length);
+          section.emit_varint(wasm.pad);
+        }
         for (let index of wasm.table) {
           section.emit_varint(index);
         }
@@ -338,19 +342,6 @@ class WasmModuleBuilder {
       });
     }
 
-    // Add an indirect function table pad section.
-    if (wasm.pad !== null) {
-      if (debug)
-        print("emitting indirect function table pad @ " + binary.length);
-      binary.emit_section(kDeclFunctionTablePad, section => {
-        section.emit_varint(wasm.pad);
-      });
-    }
-
-    // End the module.
-    if (debug) print("emitting end @ " + binary.length);
-    binary.emit_section(kDeclEnd, section => {});
-
     return binary;
   }