Patch coming.
<rdar://problem/39027923>
Created attachment 336842 [details] proposed patch.
Comment on attachment 336842 [details] proposed patch. View in context: https://bugs.webkit.org/attachment.cgi?id=336842&action=review r=me this is great! :D > Source/JavaScriptCore/wasm/js/WebAssemblyFunction.h:72 > + return m_jsEntrypoint.retagged(CodeEntryPtrTag, CodeEntryWithArityCheckPtrTag); Right now vmEntryToWebAssembly is the same thing as vmEntryToJavaScript. Should we split them and use different CodeEntryPtrTag?
Comment on attachment 336842 [details] proposed patch. View in context: https://bugs.webkit.org/attachment.cgi?id=336842&action=review Thanks for the review. >> Source/JavaScriptCore/wasm/js/WebAssemblyFunction.h:72 >> + return m_jsEntrypoint.retagged(CodeEntryPtrTag, CodeEntryWithArityCheckPtrTag); > > Right now vmEntryToWebAssembly is the same thing as vmEntryToJavaScript. Should we split them and use different CodeEntryPtrTag? Let's only make a change when the need arises later.
Landed in r230106: <http://trac.webkit.org/r230106>.