Bug 159775

Summary:

WebAssembly: support the newer spec

Product:

WebKit

Reporter:

Keith Miller <keith_miller>

Component:

JavaScriptCore

Assignee:

Keith Miller <keith_miller>

Status:

NEW

Severity:

Normal

CC:

benjamin, cdumez, chi187, cmarcelo, commit-queue, dbates, fpizlo, jfbastien, luke, mark.lam, mike, msaboff, ossy, saam, xan.lopez

Priority:

Version:

WebKit Nightly Build

Hardware:

Unspecified

OS:

Unspecified

See Also:

https://bugs.webkit.org/show_bug.cgi?id=170683

Bug Depends on:

161709, 162976, 165231, 165856, 165862, 166349, 166442, 169566, 170343, 170360, 170557, 170748, 170773, 170788, 170826, 170970, 170998, 171936, 160603, 160681, 160914, 160928, 161707, 161710, 161727, 161728, 161778, 162693, 163807, 163919, 163959, 164444, 164459, 164724, 165345, 165471, 165805, 165833, 166199, 166295, 168672, 168673, 168694, 168784, 168788, 168822, 168836, 168857, 168861, 169187, 169792, 169794, 169815, 169882, 169890, 169892, 169976, 170137, 170215, 170218, 170219, 170227, 170380, 170628, 170690, 170740, 170774, 170782, 170825, 170909, 171263, 171454, 171537, 171799, 172008

Bug Blocks:

146064

Attachments:

Description	Flags
Patch	none

Keith Miller

Reported 2016-07-14 11:47:09 PDT

We should totes do this.

Attachments
Patch (31.91 KB, patch) 2016-10-20 13:39 PDT, Keith Miller	no flags	Details Formatted Diff Diff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.

Keith Miller

Comment 1 2016-10-20 13:39:04 PDT

Created attachment 292244 [details] Patch

Keith Miller

Comment 2 2016-10-20 13:42:07 PDT

Crap, I uploaded this to the wrong bug...

Geoffrey Garen

Comment 3 2016-10-20 13:54:56 PDT

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.

JF Bastien

Comment 4 2017-01-03 20:39:10 PST

*** Bug 161728 has been marked as a duplicate of this bug. ***

Note You need to log in before you can comment on or make changes to this bug.