[wasm] Use a Managed<WasmModule> to hold metadata about modules.

src/wasm/signature-map.cc

 // Copyright 2016 the V8 project authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "src/wasm/signature-map.h"
 
 namespace v8 {
 namespace internal {
 namespace wasm {
 
@@ skipping 28 matching lines @@
     if (a->GetReturn(r) < b->GetReturn(r)) return true;
     if (a->GetReturn(r) > b->GetReturn(r)) return false;
   }
   for (size_t p = 0; p < a->parameter_count(); p++) {
     if (a->GetParam(p) < b->GetParam(p)) return true;
     if (a->GetParam(p) > b->GetParam(p)) return false;
   }
   return false;
 }
 
+bool SignatureMap::Equal(FunctionSig* a, FunctionSig* b) {
+  if (a->parameter_count() != b->parameter_count()) return false;
+  if (a->return_count() != b->return_count()) return false;
+  for (size_t i = 0; i < a->parameter_count(); i++) {

[Mircea Trofin, 2016/10/19 05:28:55]

This bottom part could be done blindly wrt what's a parameter and what's a
return value, since we 
established they are the same. At that point, can't we just compare the
underlying storage?

Any reason this API isn't a member of FunctionSig?

[titzer, 2016/10/19 09:54:41]

Done.

+    if (a->GetParam(i) != b->GetParam(i)) return false;
+  }
+  for (size_t i = 0; i < a->return_count(); i++) {
+    if (a->GetReturn(i) != b->GetReturn(i)) return false;
+  }
+  return true;
+}
+
 }  // namespace wasm
 }  // namespace internal
 }  // namespace v8