Summary: | [WASM-References] Add declared function indexes set to check from what functions we can create refs | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Dmitry <dbezhetskov> | ||||||||
Component: | WebAssembly | Assignee: | Nobody <webkit-unassigned> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Normal | CC: | chi187, ews-watchlist, keith_miller, mark.lam, msaboff, saam, tzagallo, webkit-bug-importer, ysuzuki | ||||||||
Priority: | P2 | Keywords: | InRadar | ||||||||
Version: | WebKit Nightly Build | ||||||||||
Hardware: | Unspecified | ||||||||||
OS: | Unspecified | ||||||||||
Attachments: |
|
Description
Dmitry
2020-12-18 00:10:57 PST
Created attachment 416501 [details]
Patch
Created attachment 416503 [details]
Patch
Comment on attachment 416503 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=416503&action=review > Source/JavaScriptCore/wasm/WasmModuleInformation.h:78 > + void addDeclaredFunction(uint32_t index) { m_declaredFunctions.set(index); } It could be a constant member function too probably? Comment on attachment 416503 [details]
Patch
r=me
Created attachment 416795 [details]
Patch
Comment on attachment 416503 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=416503&action=review >> Source/JavaScriptCore/wasm/WasmModuleInformation.h:78 >> + void addDeclaredFunction(uint32_t index) { m_declaredFunctions.set(index); } > > It could be a constant member function too probably? yes, but I don't truly understand why we need to declare this as const? It is a part of ModuleInfo that we change. btw, don't understand why addReferencedFunction is const too, do you know why it is written like that? View in context: https://bugs.webkit.org/attachment.cgi?id=416503&action=review >> Source/JavaScriptCore/wasm/WasmModuleInformation.h:78 >> + void addDeclaredFunction(uint32_t index) { m_declaredFunctions.set(index); } > > It could be a constant member function too probably? yes, but I don't truly understand why we need to declare this as const? It is a part of ModuleInfo that we change. btw, don't understand why addReferencedFunction is const too, do you know why it is written like that? Comment on attachment 416795 [details]
Patch
r=me if EWS gets green :)
Committed r271097: <https://trac.webkit.org/changeset/271097> All reviewed patches have been landed. Closing bug and clearing flags on attachment 416795 [details]. |