[wasm] Support recompilation if deserialization fails.

test/cctest/wasm/test-run-wasm-module.cc

 // Copyright 2015 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 <stdlib.h>
 #include <string.h>
 
 #include "src/wasm/module-decoder.h"
 #include "src/wasm/wasm-macro-gen.h"
 #include "src/wasm/wasm-module-builder.h"
@@ skipping 179 matching lines @@
   WasmFunctionBuilder* f = builder->AddFunction(sigs.i_i());
   byte code[] = {WASM_GET_LOCAL(0), kExprI32Const, 1, kExprI32Add};
   f->EmitCode(code, sizeof(code));
   ExportAs(f, kFunctionName);
 
   ZoneBuffer buffer(&zone);
   builder->WriteTo(buffer);
 
   Isolate* isolate = CcTest::InitIsolateOnce();
   ErrorThrower thrower(isolate, "");
+  uint8_t* bytes = nullptr;
+  int buffer_size = -1;
   v8::WasmCompiledModule::SerializedModule data;
   {
     HandleScope scope(isolate);
     testing::SetupIsolateForWasmModule(isolate);
 
     ModuleResult decoding_result = DecodeWasmModule(
         isolate, &zone, buffer.begin(), buffer.end(), false, kWasmOrigin);
     std::unique_ptr<const WasmModule> module(decoding_result.val);
     CHECK(!decoding_result.failed());
 
     MaybeHandle<FixedArray> compiled_module =
         module->CompileFunctions(isolate, &thrower);
     CHECK(!compiled_module.is_null());
     Handle<JSObject> module_obj = CreateCompiledModuleObject(
         isolate, compiled_module.ToHandleChecked(), ModuleOrigin::kWasmOrigin);
     v8::Local<v8::Object> v8_module_obj = v8::Utils::ToLocal(module_obj);
     CHECK(v8_module_obj->IsWebAssemblyCompiledModule());
 
     v8::Local<v8::WasmCompiledModule> v8_compiled_module =
         v8_module_obj.As<v8::WasmCompiledModule>();
+    v8::Local<v8::String> uncompiled_bytes =
+        v8_compiled_module->GetUncompiledBytes();
+    buffer_size = uncompiled_bytes->Length();
+    bytes = zone.NewArray<uint8_t>(buffer_size);
+    uncompiled_bytes->WriteOneByte(bytes);
     data = v8_compiled_module->Serialize();
   }
 
   v8::Isolate::CreateParams create_params;
   create_params.array_buffer_allocator =
       CcTest::InitIsolateOnce()->array_buffer_allocator();
 
-  v8::Isolate* v8_isolate = v8::Isolate::New(create_params);
-  {
-    v8::Isolate::Scope isolate_scope(v8_isolate);
-    v8::HandleScope new_scope(v8_isolate);
-    v8::Local<v8::Context> new_ctx = v8::Context::New(v8_isolate);
-    new_ctx->Enter();
-    isolate = reinterpret_cast<Isolate*>(v8_isolate);
-    testing::SetupIsolateForWasmModule(isolate);
-
-    v8::MaybeLocal<v8::WasmCompiledModule> deserialized =
-        v8::WasmCompiledModule::Deserialize(v8_isolate, data);
-    v8::Local<v8::WasmCompiledModule> compiled_module;
-    CHECK(deserialized.ToLocal(&compiled_module));
-    Handle<JSObject> module_object =
-        Handle<JSObject>::cast(v8::Utils::OpenHandle(*compiled_module));
-    Handle<JSObject> instance =
-        WasmModule::Instantiate(isolate, &thrower, module_object,
-                                Handle<JSReceiver>::null(),
-                                Handle<JSArrayBuffer>::null())
-            .ToHandleChecked();
-    Handle<Object> params[1] = {Handle<Object>(Smi::FromInt(41), isolate)};
-    int32_t result = testing::CallWasmFunctionForTesting(
-        isolate, instance, &thrower, kFunctionName, 1, params,
-        ModuleOrigin::kWasmOrigin);
-    CHECK(result == 42);
-    new_ctx->Exit();
+  for (int i = 0; i < 2; ++i) {
+    v8::Isolate* v8_isolate = v8::Isolate::New(create_params);
+    if (i == 1) {
+      // Mess with the serialized data to force recompilation.
+      data.first.reset();
+      data.second = 0;
+    }
+    {
+      v8::Isolate::Scope isolate_scope(v8_isolate);
+      v8::HandleScope new_scope(v8_isolate);
+      v8::Local<v8::Context> new_ctx = v8::Context::New(v8_isolate);
+      new_ctx->Enter();
+      isolate = reinterpret_cast<Isolate*>(v8_isolate);
+      testing::SetupIsolateForWasmModule(isolate);
+      Vector<const uint8_t> raw(bytes, buffer_size);
+      v8::MaybeLocal<v8::WasmCompiledModule> deserialized =
+          v8::WasmCompiledModule::DeserializeOrCompile(
+              v8_isolate, data,
+              v8::Utils::ToLocal(isolate->factory()
+                                     ->NewStringFromOneByte(raw)
+                                     .ToHandleChecked()));
+      v8::Local<v8::WasmCompiledModule> compiled_module;
+      CHECK(deserialized.ToLocal(&compiled_module));
+      Handle<JSObject> module_object =
+          Handle<JSObject>::cast(v8::Utils::OpenHandle(*compiled_module));
+      Handle<JSObject> instance =
+          WasmModule::Instantiate(isolate, &thrower, module_object,
+                                  Handle<JSReceiver>::null(),
+                                  Handle<JSArrayBuffer>::null())
+              .ToHandleChecked();
+      Handle<Object> params[1] = {Handle<Object>(Smi::FromInt(41), isolate)};
+      int32_t result = testing::CallWasmFunctionForTesting(
+          isolate, instance, &thrower, kFunctionName, 1, params,
+          ModuleOrigin::kWasmOrigin);
+      CHECK(result == 42);
+      new_ctx->Exit();
+    }
+    v8_isolate->Dispose();
   }
 }
 
 TEST(MemorySize) {
   // Initial memory size is 16, see wasm-module-builder.cc
   static const int kExpectedValue = 16;
   TestSignatures sigs;
   v8::internal::AccountingAllocator allocator;
   Zone zone(&allocator);
 
@@ skipping 250 matching lines @@
 
 TEST(Run_WasmModule_Global_f32) {
   RunWasmModuleGlobalInitTest<float>(kAstF32, -983.9f);
   RunWasmModuleGlobalInitTest<float>(kAstF32, 1122.99f);
 }
 
 TEST(Run_WasmModule_Global_f64) {
   RunWasmModuleGlobalInitTest<double>(kAstF64, -833.9);
   RunWasmModuleGlobalInitTest<double>(kAstF64, 86374.25);
 }