WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED DUPLICATE of
bug 148666
147747
jsc-tailcall: Stop saving and restoring all callee saves on vmEntry / exit for platforms with per function callee saves handling
https://bugs.webkit.org/show_bug.cgi?id=147747
Summary
jsc-tailcall: Stop saving and restoring all callee saves on vmEntry / exit fo...
Michael Saboff
Reported
2015-08-06 14:35:07 PDT
We don't need to save and restore all possible callee save registers at VM entry time for the platforms that properly handle callee save registers on function entry and exit. Currently this is just the 64 bit platforms.
Attachments
Patch
(4.68 KB, patch)
2015-08-06 14:40 PDT
,
Michael Saboff
no flags
Details
Formatted Diff
Diff
Show Obsolete
(1)
View All
Add attachment
proposed patch, testcase, etc.
Michael Saboff
Comment 1
2015-08-06 14:40:18 PDT
Created
attachment 258399
[details]
Patch
Basile Clement
Comment 2
2015-08-06 15:24:52 PDT
Comment on
attachment 258399
[details]
Patch View in context:
https://bugs.webkit.org/attachment.cgi?id=258399&action=review
LGTM
> Source/JavaScriptCore/llint/LowLevelInterpreter.asm:400 > +if C_LOOP or ARM64 or X86_64 or X86_64_WIN
I'd add a comment stating that this is handled correctly by the engine on 64 bits platforms.
Michael Saboff
Comment 3
2015-08-06 15:45:31 PDT
Committed
r188072
: <
http://trac.webkit.org/changeset/188072
>
Michael Saboff
Comment 4
2015-08-17 19:31:19 PDT
Committed
r188568
: <
http://trac.webkit.org/changeset/188568
>
Michael Saboff
Comment 5
2015-08-17 19:32:47 PDT
Relanded patch after fixes landed in change sets
r188555
&
r188556
.
Basile Clement
Comment 6
2015-08-31 18:10:42 PDT
*** This bug has been marked as a duplicate of
bug 148666
***
Csaba Osztrogonác
Comment 7
2015-09-14 10:58:56 PDT
Comment on
attachment 258399
[details]
Patch Cleared review? from
attachment 258399
[details]
so that this bug does not appear in
http://webkit.org/pending-review
. If you would like this patch reviewed, please attach it to a new bug (or re-open this bug before marking it for review 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