Chromium Code Reviews

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
Reviewers:
Mads Ager (chromium)
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 Stats (+1 line, -821 lines)
M src/arm/full-codegen-arm.cc View 2 chunks +0 lines, -2 lines 0 comments
M src/arm/ic-arm.cc View 1 chunk +0 lines, -211 lines 0 comments
M src/debug.cc View 2 chunks +0 lines, -29 lines 0 comments
M src/ia32/full-codegen-ia32.cc View 2 chunks +0 lines, -24 lines 0 comments
M src/ia32/ic-ia32.cc View 3 chunks +0 lines, -178 lines 0 comments
M src/ic.h View 8 chunks +0 lines, -47 lines 0 comments
M src/ic.cc View 7 chunks +1 line, -172 lines 0 comments
M src/x64/full-codegen-x64.cc View 2 chunks +0 lines, -24 lines 0 comments
M src/x64/ic-x64.cc View 3 chunks +0 lines, -134 lines 0 comments

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