Implement internal calls to functions that return a 32-bit integer in WebAssembly
Created attachment 260849 [details] Patch
Comment on attachment 260849 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=260849&action=review > Source/JavaScriptCore/wasm/WASMFunctionCompiler.h:685 > + Vector<CallCompilationInfo> m_callCompilationInfo; You need to do something to GC this correctly. See our other handling of CallLinkInfo.
Comment on attachment 260849 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=260849&action=review >> Source/JavaScriptCore/wasm/WASMFunctionCompiler.h:685 >> + Vector<CallCompilationInfo> m_callCompilationInfo; > > You need to do something to GC this correctly. See our other handling of CallLinkInfo. I think he's doing it right already. The CallLinkInfos get added to a CodeBlock. The CodeBlock then knows what to do.
Comment on attachment 260849 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=260849&action=review > Source/JavaScriptCore/wasm/WASMFunctionCompiler.h:642 > + CallLinkInfo* info = m_codeBlock->addCallLinkInfo(); I see. Is good.
Comment on attachment 260849 [details] Patch LGTM.
Comment on attachment 260849 [details] Patch Clearing flags on attachment: 260849 Committed r189563: <http://trac.webkit.org/changeset/189563>
All reviewed patches have been landed. Closing bug.
Comment on attachment 260849 [details] Patch r=me