Bug 122487

Summary: FTL should use llvm.webkit.stackmap for OSR exit
Product: WebKit Reporter: Filip Pizlo <fpizlo>
Component: JavaScriptCoreAssignee: Filip Pizlo <fpizlo>
Status: RESOLVED FIXED    
Severity: Normal CC: atrick, barraclough, ggaren, mark.lam, mhahnenberg, msaboff, nrotem, oliver, sam
Priority: P2 Keywords: InRadar
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Bug Depends on: 122518, 122538, 122666, 122688    
Bug Blocks: 122318    
Attachments:
Description Flags
work in progress
none
more! none

Filip Pizlo
Reported 2013-10-07 22:01:59 PDT
Patch forthcoming. This will take some iterating to get right.
Attachments
work in progress (23.25 KB, patch)
2013-10-07 22:02 PDT, Filip Pizlo 		no flags
DetailsFormatted DiffDiff
more! (33.59 KB, patch)
2013-10-08 12:40 PDT, Filip Pizlo 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Filip Pizlo
Comment 1 2013-10-07 22:02:26 PDT
Created attachment 213652 [details] work in progress
Filip Pizlo
Comment 2 2013-10-08 12:40:38 PDT
Created attachment 213707 [details] more!
Filip Pizlo
Comment 3 2013-10-08 20:25:17 PDT
<rdar://problem/15183189>
Filip Pizlo
Comment 4 2013-10-13 21:54:08 PDT
Leaving this open until the LLVM side of the patches lands and we can enable this by default.
Filip Pizlo
Comment 5 2013-10-17 23:15:22 PDT
This works reliably as of http://trac.webkit.org/changeset/157382, but I'll leave it open until the functionality is available in LLVM trunk.
Filip Pizlo
Comment 6 2013-11-02 20:13:55 PDT
It's in trunk. The remaining work to port the implementation to the slight changes between the original LLVM patch and what landed will be covered in https://bugs.webkit.org/show_bug.cgi?id=122318.
Note You need to log in before you can comment on or make changes to this bug.