WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
129810
FTL arity fixup should work on ARM64
https://bugs.webkit.org/show_bug.cgi?id=129810
Summary
FTL arity fixup should work on ARM64
Filip Pizlo
Reported
2014-03-06 11:41:27 PST
Patch forthcoming.
Attachments
the patch
(10.08 KB, patch)
2014-03-06 11:44 PST
,
Filip Pizlo
msaboff
: review+
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
Filip Pizlo
Comment 1
2014-03-06 11:44:23 PST
Created
attachment 226021
[details]
the patch
Michael Saboff
Comment 2
2014-03-06 11:53:48 PST
Comment on
attachment 226021
[details]
the patch r=me Did you test on X86-{32,64}?
Filip Pizlo
Comment 3
2014-03-06 11:55:11 PST
(In reply to
comment #2
)
> (From update of
attachment 226021
[details]
) > r=me > > Did you test on X86-{32,64}?
X86-64 yes. I'm still testing 32-bit.
Filip Pizlo
Comment 4
2014-03-06 12:11:22 PST
Landed in
http://trac.webkit.org/changeset/165205
WebKit Commit Bot
Comment 5
2014-03-06 12:26:46 PST
Re-opened since this is blocked by
bug 129813
Brent Fulgham
Comment 6
2014-03-06 14:59:54 PST
It looks like this bug broke the Windows build: 1>LowLevelInterpreterWin.obj : error LNK2026: module unsafe for SAFESEH image. Creating library C:/cygwin/home/buildbot/slave/windows-release-archive/build/build-Release\Production\lib32\JavaScriptCore.lib and object C:/cygwin/home/buildbot/slave/windows-release-archive/build/build-Release\Production\lib32\JavaScriptCore.exp 1>C:/cygwin/home/buildbot/slave/windows-release-archive/build/build-Release\Production\bin32\JavaScriptCore.dll : fatal error LNK1281: Unable to generate SAFESEH image. This may be due to adding the 'raise' handler to the 'when "t7"' clause.
Brent Fulgham
Comment 7
2014-03-06 15:00:18 PST
http://msdn.microsoft.com/en-us/library/9a89h429.aspx
Filip Pizlo
Comment 8
2014-03-06 20:56:49 PST
This wasn't rolled out.
Filip Pizlo
Comment 9
2014-03-06 20:58:39 PST
(In reply to
comment #7
)
>
http://msdn.microsoft.com/en-us/library/9a89h429.aspx
Are we using cloop or the fully assembly llint on Windows? If the latter, then someone just needs to play with register mappings. It would be fine to have different register mappings on Windows and Darwin+Linux. It would not be acceptable to change the mappings on Darwin+Linux at this point, since they are based on the SysV x86-64 ABI and we have to obey this ABI.
Mark Lam
Comment 10
2014-03-06 21:05:18 PST
(In reply to
comment #9
)
> (In reply to
comment #7
) > >
http://msdn.microsoft.com/en-us/library/9a89h429.aspx
> > Are we using cloop or the fully assembly llint on Windows? > > If the latter, then someone just needs to play with register mappings. It would be fine to have different register mappings on Windows and Darwin+Linux. It would not be acceptable to change the mappings on Darwin+Linux at this point, since they are based on the SysV x86-64 ABI and we have to obey this ABI.
FYI, we now use the ASM LLINT and also enable the baseline JIT on Windows with recent patches by peavo.
Mark Lam
Comment 11
2014-03-06 21:10:54 PST
(In reply to
comment #6
)
> 1>C:/cygwin/home/buildbot/slave/windows-release-archive/build/build-Release\Production\bin32\JavaScriptCore.dll : fatal error LNK1281: Unable to generate SAFESEH image.
Just to clarify, it turned out that the “SAFESEH” issue is not due to this patch. Instead, the issue started manifesting with the patch for
http://trac.webkit.org/changeset/165128
. It just needed some additional build rules in the MSVC project. Brent added those rules this afternoon, and all is well again.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug