Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(391)

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: src/wasm/wasm-objects.cc

Issue 2807013002: [wasm] Misc fixes for async compilation (Closed)
Patch Set: [wasm] Fix async compile for empty modules, or modules with tables. Created 3 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« src/wasm/wasm-module.cc ('K') | « src/wasm/wasm-module.cc ('k') | test/mjsunit/regress/wasm/regress-709684.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/wasm/wasm-objects.cc
diff --git a/src/wasm/wasm-objects.cc b/src/wasm/wasm-objects.cc
index cbab5e7dcd04d91d77fb77fc5312b43f41c15732..da8d4a88cee7c121aa7d81a953227ff5be203d1b 100644
--- a/src/wasm/wasm-objects.cc
+++ b/src/wasm/wasm-objects.cc
@@ -967,6 +967,8 @@ Handle<WasmCompiledModule> WasmCompiledModule::New(
maybe_signature_tables.ToHandleChecked());
compiled_module->set_empty_function_tables(
maybe_empty_function_tables.ToHandleChecked());
+ compiled_module->set_function_tables(
+ maybe_empty_function_tables.ToHandleChecked());
}
// TODO(mtrofin): we copy these because the order of finalization isn't
// reliable, and we need these at Reset (which is called at
« src/wasm/wasm-module.cc ('K') | « src/wasm/wasm-module.cc ('k') | test/mjsunit/regress/wasm/regress-709684.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698