Bug 277590

Summary: [JSC] Update outdated comments for `WasmModuleInformation`
Product: WebKit Reporter: Sosuke Suzuki <aosukeke>
Component: JavaScriptCoreAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   

Sosuke Suzuki
Reported 2024-08-03 10:19:36 PDT
The comments in `WasmModuleInformat.h` state that the JSC Wasm implementation can only have one memory and one table. However, https://github.com/WebKit/WebKit/commit/f8e5d3c58c7eafc12c1264c2321e72ec67a6077a now allows for multiple tables. Therefore, these comments are outdated.
Attachments
Add attachment proposed patch, testcase, etc.
Sosuke Suzuki
Comment 1 2024-08-03 10:20:30 PDT
Pull request: https://github.com/WebKit/WebKit/pull/31691
EWS
Comment 2 2024-08-04 03:15:23 PDT
Committed 281815@main (23c76f82dce2): <https://commits.webkit.org/281815@main> Reviewed commits have been landed. Closing PR #31691 and removing active labels.
Radar WebKit Bug Importer
Comment 3 2024-08-04 03:16:14 PDT
<rdar://problem/133152812>
Note You need to log in before you can comment on or make changes to this bug.