[webkit-changes] [WebKit/WebKit] fc9e1f: [Wasm-GC] Fix initialization of portable reftype g...
Asumu Takikawa
noreply at github.com
Fri Dec 8 18:09:10 PST 2023
Branch: refs/heads/main
Home: https://github.com/WebKit/WebKit
Commit: fc9e1f45e692511bdf47f9a49102cdeadcc6e707
https://github.com/WebKit/WebKit/commit/fc9e1f45e692511bdf47f9a49102cdeadcc6e707
Author: Asumu Takikawa <asumu at igalia.com>
Date: 2023-12-08 (Fri, 08 Dec 2023)
Changed paths:
M JSTests/wasm/gc/js-api.js
M Source/JavaScriptCore/wasm/WasmGlobal.cpp
M Source/JavaScriptCore/wasm/js/WebAssemblyModuleRecord.cpp
Log Message:
-----------
[Wasm-GC] Fix initialization of portable reftype globals
https://bugs.webkit.org/show_bug.cgi?id=265693
Reviewed by Justin Michaud.
Ref-typed globals need to be initialized using the JSValue init path rather
than the "as bits" init path. This is less a problem for functions, as they are
held strongly by the instance, but for other GC types is a bigger problem.
For portable globals, the marking method also needs to check for all reftypes,
not just extern/func.
This is mostly tested by the existing gc/js-api.js test, one test was added for
the marking issue.
* JSTests/wasm/gc/js-api.js:
* Source/JavaScriptCore/wasm/WasmGlobal.cpp:
(JSC::Wasm::Global::visitAggregateImpl):
* Source/JavaScriptCore/wasm/js/WebAssemblyModuleRecord.cpp:
(JSC::WebAssemblyModuleRecord::initializeExports):
Canonical link: https://commits.webkit.org/271777@main
More information about the webkit-changes
mailing list