Bug 127228

Summary: CStack: Switching Windows port to using the C Loop LLINT
Product: WebKit Reporter: Mark Lam <mark.lam>
Component: JavaScriptCoreAssignee: Mark Lam <mark.lam>
Status: RESOLVED FIXED    
Severity: Normal CC: achristensen, bfulgham, fpizlo, ggaren, mhahnenberg, msaboff, oliver, roger_fong
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
the patch. ggaren: review+

Mark Lam
Reported 2014-01-18 11:26:54 PST
With the switch to using the C stack, the LLINT becomes a mandatory component for providing trampoline glue code to enter JIT code. Since the Windows port does not currently have a LLINT implementation that can generate assembly code for Windows, we’ll switch the Windows port to using the C Loop LLINT.
Attachments
the patch. (19.31 KB, patch)
2014-01-18 11:45 PST, Mark Lam 		ggaren: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Mark Lam
Comment 1 2014-01-18 11:45:03 PST
Created attachment 221562 [details] the patch. This patch is not tested on a Windows build yet. Will test before landing.
Geoffrey Garen
Comment 2 2014-01-18 12:37:04 PST
Comment on attachment 221562 [details] the patch. View in context: https://bugs.webkit.org/attachment.cgi?id=221562&action=review r=me > Source/JavaScriptCore/ChangeLog:11 > + can generate assembly code for Windows, weâll switch the Windows port to ASCII text for "we'll", please.
Mark Lam
Comment 3 2014-01-22 17:58:51 PST
The patch builds on Windows. Landed in r162577: <http://trac.webkit.org/r162577>.
Note You need to log in before you can comment on or make changes to this bug.