Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(353)

Issue 1327093002: PPC: [builtins] Unify the various versions of [[Call]] with a Call builtin. (Closed)

Created:
5 years, 3 months ago by MTBrandyberry
Modified:
5 years, 3 months ago
CC:
v8-dev
Base URL:
https://chromium.googlesource.com/v8/v8.git@master
Target Ref:
refs/pending/heads/master
Project:
v8
Visibility:
Public.

Description

PPC: [builtins] Unify the various versions of [[Call]] with a Call builtin. Port ccbb4ff00f1d8f32fd9227cd7aba1723791e5744 Original commit message: The new Call and CallFunction builtins supersede the current CallFunctionStub (and CallIC magic) and will be the single bottleneck for all calling, including the currently special Function.prototype.call and Function.prototype.apply builtins, which had handwritten (and not fully compliant) versions of CallFunctionStub, and also the CallIC(s), which where also slightly different. This also reduces the overhead for API function calls, which is still unnecessary high, but let's do that step-by-step. This also fixes a bunch of cases where the implicit ToObject for sloppy receivers was done in the wrong context (in the caller context instead of the callee context), which basically meant that we allowed cross context access to %ObjectPrototype%. MIPS and MIPS64 ports contributed by akos.palfi@imgtec.com. R=bmeurer@chromium.org, joransiu@ca.ibm.com, jyan@ca.ibm.com, michael_dawson@ca.ibm.com, dstence@us.ibm.com BUG=v8:4413 LOG=n Committed: https://crrev.com/31a9396e4b4a6fe71f238d248400d4c1e5e27258 Cr-Commit-Position: refs/heads/master@{#30656}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+209 lines, -296 lines) Patch
M src/ppc/builtins-ppc.cc View 5 chunks +160 lines, -262 lines 0 comments Download
M src/ppc/code-stubs-ppc.cc View 7 chunks +11 lines, -34 lines 0 comments Download
M src/ppc/macro-assembler-ppc.h View 3 chunks +21 lines, -0 lines 0 comments Download
M src/ppc/macro-assembler-ppc.cc View 2 chunks +17 lines, -0 lines 0 comments Download

Messages

Total messages: 6 (1 generated)
MTBrandyberry
5 years, 3 months ago (2015-09-08 22:25:52 UTC) #1
dstence
lgtm
5 years, 3 months ago (2015-09-09 13:58:45 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1327093002/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1327093002/1
5 years, 3 months ago (2015-09-09 14:04:23 UTC) #4
commit-bot: I haz the power
Committed patchset #1 (id:1)
5 years, 3 months ago (2015-09-09 14:39:58 UTC) #5
commit-bot: I haz the power
5 years, 3 months ago (2015-09-09 14:40:23 UTC) #6
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/31a9396e4b4a6fe71f238d248400d4c1e5e27258
Cr-Commit-Position: refs/heads/master@{#30656}

Powered by Google App Engine
This is Rietveld 408576698