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

Issue 12374044: Unify deoptimizer for accessor and arguments frames. (Closed)

Created:
7 years, 9 months ago by Michael Starzinger
Modified:
7 years, 9 months ago
Reviewers:
Sven Panne
CC:
v8-dev
Visibility:
Public.

Description

Unify deoptimizer for accessor and arguments frames. This unifies the translation of artificial accessor stub and arguments adaptor frames. The frame layout is the same on all architectures and the computation code can be shared. R=svenpanne@chromium.org Committed: http://code.google.com/p/v8/source/detail?r=13784

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+220 lines, -863 lines) Patch
M src/arm/deoptimizer-arm.cc View 2 chunks +0 lines, -214 lines 0 comments Download
M src/deoptimizer.cc View 1 chunk +220 lines, -0 lines 0 comments Download
M src/ia32/deoptimizer-ia32.cc View 2 chunks +0 lines, -215 lines 0 comments Download
M src/mips/deoptimizer-mips.cc View 2 chunks +0 lines, -214 lines 0 comments Download
M src/x64/deoptimizer-x64.cc View 2 chunks +0 lines, -220 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
Michael Starzinger
This first step is purely mechanical.
7 years, 9 months ago (2013-03-01 11:33:12 UTC) #1
Sven Panne
lgtm
7 years, 9 months ago (2013-03-01 12:21:35 UTC) #2
Michael Starzinger
7 years, 9 months ago (2013-03-01 12:23:35 UTC) #3
Message was sent while issue was closed.
Committed patchset #1 manually as r13784 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698