Remove FunctionType (#2510)

Function signatures were previously redundantly stored on Function
objects as well as on FunctionType objects. These two signature
representations had to always be kept in sync, which was error-prone
and needlessly complex. This PR takes advantage of the new ability of
Type to represent multiple value types by consolidating function
signatures as a pair of Types (params and results) stored on the
Function object.

Since there are no longer module-global named function types,
significant changes had to be made to the printing and emitting of
function types, as well as their parsing and manipulation in various
passes.

The C and JS APIs and their tests also had to be updated to remove
named function types.

1 files changed, 1 insertions, 5 deletions

diff --git a/test/binaryen.js/functions.js b/test/binaryen.js/functions.js
index cd821dc96..b2172d005 100644
--- a/test/binaryen.js/functions.js
+++ b/test/binaryen.js/functions.js
@@ -14,9 +14,7 @@ function cleanInfo(info) {
 
 var module = new Binaryen.Module();
 
-var signature = module.addFunctionType("i", Binaryen.i32, []);
-
-var func = module.addFunction("a-function", signature, [],
+var func = module.addFunction("a-function", Binaryen.none, Binaryen.i32, [],
   module.i32.add(
     module.i32.const(1),
     module.i32.const(2)
@@ -27,8 +25,6 @@ console.log("GetFunction is equal: " + (func === module.getFunction("a-function"
 
 module.runPassesOnFunction(func, ["precompute"]);
 
-var sigInfo = Binaryen.getFunctionTypeInfo(signature);
-console.log("getFunctionTypeInfo=" + JSON.stringify(cleanInfo(sigInfo)));
 var funcInfo = Binaryen.getFunctionInfo(func);
 console.log("getFunctionInfo=" + JSON.stringify(cleanInfo(funcInfo)));
 var expInfo = Binaryen.getExpressionInfo(funcInfo.body);