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

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

Issue 2651903002: [wasm] No need to use multiple inheritance for `ModuleBytesEnv` (Closed)
Patch Set: Created 3 years, 11 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
« no previous file with comments | « src/wasm/module-decoder.cc ('k') | src/wasm/wasm-module.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/wasm/wasm-interpreter.cc
diff --git a/src/wasm/wasm-interpreter.cc b/src/wasm/wasm-interpreter.cc
index fbcfea819638da30fd4c8906d24db4817290d4f6..9a23b748e13b9ec2b7a1446e186560f43b40cff1 100644
--- a/src/wasm/wasm-interpreter.cc
+++ b/src/wasm/wasm-interpreter.cc
@@ -1784,12 +1784,13 @@ class WasmInterpreterInternals : public ZoneObject {
ZoneVector<ThreadImpl> threads_;
WasmInterpreterInternals(Zone* zone, const ModuleBytesEnv& env)
- : instance_(env.instance),
- module_bytes_(env.module_bytes.start(), env.module_bytes.end(), zone),
- codemap_(env.instance ? env.instance->module : nullptr,
- module_bytes_.data(), zone),
+ : instance_(env.module_env.instance),
+ module_bytes_(env.wire_bytes.start(), env.wire_bytes.end(), zone),
+ codemap_(
+ env.module_env.instance ? env.module_env.instance->module : nullptr,
+ module_bytes_.data(), zone),
threads_(zone) {
- threads_.emplace_back(zone, &codemap_, env.instance);
+ threads_.emplace_back(zone, &codemap_, env.module_env.instance);
}
void Delete() { threads_.clear(); }
« no previous file with comments | « src/wasm/module-decoder.cc ('k') | src/wasm/wasm-module.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698