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

Unified Diff: test/mjsunit/wasm/compiled-module-management.js

Issue 2306403002: Revert of [wasm] reuse the first compiled module (Closed)
Patch Set: Created 4 years, 3 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/wasm-module.cc ('k') | test/mjsunit/wasm/instantiate-module-basic.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: test/mjsunit/wasm/compiled-module-management.js
diff --git a/test/mjsunit/wasm/compiled-module-management.js b/test/mjsunit/wasm/compiled-module-management.js
deleted file mode 100644
index 5b64e00f4139e8993ed28ce62a8b4aab62aa3c5a..0000000000000000000000000000000000000000
--- a/test/mjsunit/wasm/compiled-module-management.js
+++ /dev/null
@@ -1,50 +0,0 @@
-// 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.
-
-// TODO (mtrofin): re-enable ignition (v8:5345)
-// Flags: --no-ignition --no-ignition-staging
-// Flags: --expose-wasm --expose-gc --allow-natives-syntax
-
-load("test/mjsunit/wasm/wasm-constants.js");
-load("test/mjsunit/wasm/wasm-module-builder.js");
-
-
-(function CompiledModuleInstancesAreGCed() {
- var builder = new WasmModuleBuilder();
-
- builder.addMemory(1,1, true);
- builder.addImport("getValue", kSig_i);
- builder.addFunction("f", kSig_i)
- .addBody([
- kExprCallImport, kArity0, 0
- ]).exportFunc();
-
- var module = new WebAssembly.Module(builder.toBuffer());
- %ValidateWasmModuleState(module);
- %ValidateWasmInstancesChain(module, 0);
- var i1 = new WebAssembly.Instance(module, {getValue: () => 1});
- %ValidateWasmInstancesChain(module, 1);
- var i2 = new WebAssembly.Instance(module, {getValue: () => 2});
- %ValidateWasmInstancesChain(module, 2);
- var i3 = new WebAssembly.Instance(module, {getValue: () => 3});
- %ValidateWasmInstancesChain(module, 3);
-
- assertEquals(1, i1.exports.f());
- i1 = null;
- gc();
- %ValidateWasmInstancesChain(module, 2);
- assertEquals(3, i3.exports.f());
- i3 = null;
- gc();
- %ValidateWasmInstancesChain(module, 1);
- assertEquals(2, i2.exports.f());
- i2 = null;
- gc();
- %ValidateWasmModuleState(module);
- var i4 = new WebAssembly.Instance(module, {getValue: () => 4});
- assertEquals(4, i4.exports.f());
- module = null;
- gc();
- %ValidateWasmOrphanedInstance(i4);
-})();
« no previous file with comments | « src/wasm/wasm-module.cc ('k') | test/mjsunit/wasm/instantiate-module-basic.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698