Bug 122487 - FTL should use llvm.webkit.stackmap for OSR exit
Summary: FTL should use llvm.webkit.stackmap for OSR exit
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: JavaScriptCore (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Filip Pizlo
URL:
Keywords: InRadar
Depends on: 122518 122538 122666 122688
Blocks: 122318
  Show dependency tree / graph
 
Reported: 2013-10-07 22:01 PDT by Filip Pizlo
Modified: 2013-11-02 20:13 PDT (History)
9 users (show)

See Also:

Attachments
work in progress (23.25 KB, patch)
2013-10-07 22:02 PDT, Filip Pizlo 		no flags Details | Formatted Diff | Diff
more! (33.59 KB, patch)
2013-10-08 12:40 PDT, Filip Pizlo 		no flags Details | Formatted Diff | Diff
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Filip Pizlo 2013-10-07 22:01:59 PDT 
Patch forthcoming.

This will take some iterating to get right.
Comment 1 Filip Pizlo 2013-10-07 22:02:26 PDT 
Created attachment 213652 [details]
work in progress
Comment 2 Filip Pizlo 2013-10-08 12:40:38 PDT 
Created attachment 213707 [details]
more!
Comment 3 Filip Pizlo 2013-10-08 20:25:17 PDT 
<rdar://problem/15183189>
Comment 4 Filip Pizlo 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.
Comment 5 Filip Pizlo 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.
Comment 6 Filip Pizlo 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.