[wasm] Use WasmCompiledModule type in interfaces

src/runtime/runtime-test.cc

 // Copyright 2014 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/runtime/runtime-utils.h"
 
 #include <memory>
 
 #include "src/arguments.h"
 #include "src/compiler-dispatcher/optimizing-compile-dispatcher.h"
@@ skipping 745 matching lines @@
 RUNTIME_FUNCTION(Runtime_DeserializeWasmModule) {
   HandleScope shs(isolate);
   DCHECK(args.length() == 1);
   CONVERT_ARG_HANDLE_CHECKED(JSArrayBuffer, buffer, 0);
 
   Address mem_start = static_cast<Address>(buffer->backing_store());
   int mem_size = static_cast<int>(buffer->byte_length()->Number());
 
   ScriptData sc(mem_start, mem_size);
   MaybeHandle<FixedArray> maybe_compiled_module =
       WasmCompiledModuleSerializer::DeserializeWasmModule(isolate, &sc);

[ahaas, 2016/10/11 07:46:05]

Couldn't DeserializeWasmModule already return a wasm::WasmCompiledModule?

[Clemens Hammacher, 2016/10/11 08:19:19]

We would have to include wasm headers in code-serializer.h, which I did not want
to do if not really necessary.

   Handle<FixedArray> compiled_module;
   if (!maybe_compiled_module.ToHandle(&compiled_module)) {
     return isolate->heap()->undefined_value();
   }
-  return *wasm::CreateCompiledModuleObject(isolate, compiled_module,
-                                           wasm::ModuleOrigin::kWasmOrigin);
+  return *wasm::CreateCompiledModuleObject(
+      isolate, Handle<wasm::WasmCompiledModule>::cast(compiled_module),
+      wasm::kWasmOrigin);
 }
 
 RUNTIME_FUNCTION(Runtime_ValidateWasmInstancesChain) {
   HandleScope shs(isolate);
   DCHECK(args.length() == 2);
   CONVERT_ARG_HANDLE_CHECKED(JSObject, module_obj, 0);
   CONVERT_ARG_HANDLE_CHECKED(Smi, instance_count, 1);
   wasm::testing::ValidateInstancesChain(isolate, module_obj,
                                         instance_count->value());
   return isolate->heap()->ToBoolean(true);
@@ skipping 10 matching lines @@
 RUNTIME_FUNCTION(Runtime_ValidateWasmOrphanedInstance) {
   HandleScope shs(isolate);
   DCHECK(args.length() == 1);
   CONVERT_ARG_HANDLE_CHECKED(JSObject, instance_obj, 0);
   wasm::testing::ValidateOrphanedInstance(isolate, instance_obj);
   return isolate->heap()->ToBoolean(true);
 }
 
 }  // namespace internal
 }  // namespace v8