WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
123844
Change ctiTrampoline into a thunk
https://bugs.webkit.org/show_bug.cgi?id=123844
Summary
Change ctiTrampoline into a thunk
Michael Saboff
Reported
2013-11-05 17:53:32 PST
Given that moving JSC to the C++ stack will require changes to all flavors of ctiTrampoline, it makes sense to convert it and its companion, ctiOpThrowNotCaught, to a thunks.
Attachments
Patch
(59.08 KB, patch)
2013-11-05 18:47 PST
,
Michael Saboff
gtk-ews
: commit-queue-
Details
Formatted Diff
Diff
Updated with speculative build fixes
(66.48 KB, patch)
2013-11-05 21:22 PST
,
Michael Saboff
fpizlo
: review+
eflews.bot
: commit-queue-
Details
Formatted Diff
Diff
Show Obsolete
(1)
View All
Add attachment
proposed patch, testcase, etc.
Michael Saboff
Comment 1
2013-11-05 18:47:42 PST
Created
attachment 216107
[details]
Patch The changes for ARM traditional, ARM64, MIPS, SH4 and Windows are speculative. The EWS bots will test some of these.
kov's GTK+ EWS bot
Comment 2
2013-11-05 19:05:12 PST
Comment on
attachment 216107
[details]
Patch
Attachment 216107
[details]
did not pass gtk-ews (gtk): Output:
http://webkit-queues.appspot.com/results/21248083
Michael Saboff
Comment 3
2013-11-05 21:22:37 PST
Created
attachment 216122
[details]
Updated with speculative build fixes
EFL EWS Bot
Comment 4
2013-11-05 21:40:24 PST
Comment on
attachment 216122
[details]
Updated with speculative build fixes
Attachment 216122
[details]
did not pass efl-wk2-ews (efl-wk2): Output:
http://webkit-queues.appspot.com/results/21668004
Filip Pizlo
Comment 5
2013-11-06 08:39:44 PST
Comment on
attachment 216122
[details]
Updated with speculative build fixes Cool!
Michael Saboff
Comment 6
2013-11-06 09:27:58 PST
Committed
r158751
: <
http://trac.webkit.org/changeset/158751
>
Michael Saboff
Comment 7
2013-11-20 10:17:59 PST
***
Bug 123908
has been marked as a duplicate of this bug. ***
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