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, 2 insertions, 2 deletions

diff --git a/src/tools/execution-results.h b/src/tools/execution-results.h
index 9ef3d2e1e..c0c7428cc 100644
--- a/src/tools/execution-results.h
+++ b/src/tools/execution-results.h
@@ -67,7 +67,7 @@ struct ExecutionResults {
         }
         std::cout << "[fuzz-exec] calling " << exp->name << "\n";
         auto* func = wasm.getFunction(exp->value);
-        if (func->result != none) {
+        if (func->sig.results != Type::none) {
           // this has a result
           results[exp->name] = run(func, wasm, instance);
           // ignore the result if we hit an unreachable and returned no value
@@ -136,7 +136,7 @@ struct ExecutionResults {
         instance.callFunction(ex->value, arguments);
       }
       // call the method
-      for (Type param : func->params) {
+      for (Type param : func->sig.params.expand()) {
         // zeros in arguments TODO: more?
         arguments.push_back(Literal(param));
       }