Bug 170244

Summary: WebAssembly: pack OpcodeOrigin to fit in a pointer
Product: WebKit Reporter: Keith Miller <keith_miller>
Component: New BugsAssignee: Keith Miller <keith_miller>
Status: RESOLVED FIXED    
Severity: Normal CC: buildbot, mark.lam, msaboff, saam
Priority: P2    
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch msaboff: review+

Keith Miller
Reported 2017-03-29 10:08:40 PDT
WebAssembly: pack OpcodeOrigin to fit in a pointer
Attachments
Patch (4.98 KB, patch)
2017-03-29 10:15 PDT, Keith Miller 		msaboff: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Keith Miller
Comment 1 2017-03-29 10:15:09 PDT
Created attachment 305753 [details] Patch
Michael Saboff
Comment 2 2017-03-29 10:23:55 PDT
Comment on attachment 305753 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=305753&action=review r=me > Source/JavaScriptCore/ChangeLog:8 > + This makes it so we don't have to have different allocate the OpcodeOrigin for B3. Think this sentence needs a little work.
Keith Miller
Comment 3 2017-03-29 10:27:30 PDT
Committed r214544: <http://trac.webkit.org/changeset/214544>
Keith Miller
Comment 4 2017-03-29 10:29:00 PDT
Comment on attachment 305753 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=305753&action=review >> Source/JavaScriptCore/ChangeLog:8 >> + This makes it so we don't have to have different allocate the OpcodeOrigin for B3. > > Think this sentence needs a little work. I changed it to: This patch makes it so we don't have to have allocate OpcodeOrigins and can just pack all the data into the pointer B3::Origin already has.
Note You need to log in before you can comment on or make changes to this bug.