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

Issue 6850015: Remove support for inlined property loads and stores. (Closed)

Created:
9 years, 8 months ago by fschneider
Modified:
9 years, 6 months ago
CC:
v8-dev
Visibility:
Public.

Description

Remove support for inlined property loads and stores. The full code generator does not generate inline code for property loads and stores. All this code is unused with Crankshaft. Committed: http://code.google.com/p/v8/source/detail?r=7623

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -821 lines) Patch
M src/arm/full-codegen-arm.cc View 2 chunks +0 lines, -2 lines 0 comments Download
M src/arm/ic-arm.cc View 1 chunk +0 lines, -211 lines 0 comments Download
M src/debug.cc View 2 chunks +0 lines, -29 lines 0 comments Download
M src/ia32/full-codegen-ia32.cc View 2 chunks +0 lines, -24 lines 0 comments Download
M src/ia32/ic-ia32.cc View 3 chunks +0 lines, -178 lines 0 comments Download
M src/ic.h View 8 chunks +0 lines, -47 lines 0 comments Download
M src/ic.cc View 7 chunks +1 line, -172 lines 0 comments Download
M src/x64/full-codegen-x64.cc View 2 chunks +0 lines, -24 lines 0 comments Download
M src/x64/ic-x64.cc View 3 chunks +0 lines, -134 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
fschneider
Weeding out more unused code.
9 years, 8 months ago (2011-04-14 15:46:13 UTC) #1
Mads Ager (chromium)
9 years, 8 months ago (2011-04-15 07:43:31 UTC) #2
LGTM! Greatness!

Powered by Google App Engine
This is Rietveld 408576698