Bug 147747 - jsc-tailcall: Stop saving and restoring all callee saves on vmEntry / exit for platforms with per function callee saves handling
Summary: jsc-tailcall: Stop saving and restoring all callee saves on vmEntry / exit fo...
Status: RESOLVED DUPLICATE of bug 148666
Alias: None
Product: WebKit
Classification: Unclassified
Component: JavaScriptCore (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Michael Saboff
Depends on: 147639 147759
Blocks: 146845
  Show dependency treegraph
Reported: 2015-08-06 14:35 PDT by Michael Saboff
Modified: 2015-09-14 10:58 PDT (History)
1 user (show)

See Also:

Patch (4.68 KB, patch)
2015-08-06 14:40 PDT, Michael Saboff
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Michael Saboff 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.
Comment 1 Michael Saboff 2015-08-06 14:40:18 PDT
Created attachment 258399 [details]
Comment 2 Basile Clement 2015-08-06 15:24:52 PDT
Comment on attachment 258399 [details]

View in context: https://bugs.webkit.org/attachment.cgi?id=258399&action=review


> 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.
Comment 3 Michael Saboff 2015-08-06 15:45:31 PDT
Committed r188072: <http://trac.webkit.org/changeset/188072>
Comment 4 Michael Saboff 2015-08-17 19:31:19 PDT
Committed r188568: <http://trac.webkit.org/changeset/188568>
Comment 5 Michael Saboff 2015-08-17 19:32:47 PDT
Relanded patch after fixes landed in change sets r188555 & r188556.
Comment 6 Basile Clement 2015-08-31 18:10:42 PDT

*** This bug has been marked as a duplicate of bug 148666 ***
Comment 7 Csaba Osztrogonác 2015-09-14 10:58:56 PDT
Comment on attachment 258399 [details]

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).