We should totes do this.
Created attachment 292244 [details] Patch
Crap, I uploaded this to the wrong bug...
Comment on attachment 292244 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=292244&action=review > Source/JavaScriptCore/ChangeLog:12 > + address we need to call to. For each callsite in the compiling function WASM remembers what remembers what => remembers > Source/JavaScriptCore/ChangeLog:14 > + each functions callsites are linked to the appropriate entrypoint. each callsite is linked to the appropriate entrypoint > Source/JavaScriptCore/wasm/WASMB3IRGenerator.cpp:220 > + Vector<UnlinkedLocationInfo>& m_unlinkedCalls; UnlinkedLocationInfo holds a single CodeLocationCall. What's the "list of of callsites"? > Source/JavaScriptCore/wasm/WASMB3IRGenerator.cpp:588 > + *callSiteLinkLocation = { linkBuffer.locationOf(call), functionIndex }; If m_unlinkedCalls grows enough before this patchpoint generator runs, callSiteLinkLocation becomes a stale pointer. I think you want an index instead of a pointer. > Source/JavaScriptCore/wasm/WASMFormat.h:131 > +struct UnlinkedLocationInfo { Can we call this UnlinkedCall or UnlinkedWASMCall or UnlinkedCallInfo? 'Location' is pretty vague.
*** Bug 161728 has been marked as a duplicate of this bug. ***