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

Issue 7484022: Do not explicitly record undetectable objects in the ToBoolean stub. (Closed)

Created:
9 years, 5 months ago by Sven Panne
Modified:
9 years, 5 months ago
Reviewers:
fschneider, danno
CC:
v8-dev
Visibility:
Public.

Description

Do not explicitly record undetectable objects in the ToBoolean stub. Loading the map within the stub already implies a check for an undetectable object, so there is no need to record this separately. Furthermore, this brings the size of the type info to record down to 8 bits, removing the need to find a place for the ninth bit in the Code object. ;-) Committed: http://code.google.com/p/v8/source/detail?r=8722

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+24 lines, -26 lines) Patch
M src/code-stubs.h View 3 chunks +12 lines, -5 lines 0 comments Download
M src/code-stubs.cc View 2 chunks +5 lines, -8 lines 0 comments Download
M src/ia32/code-stubs-ia32.cc View 3 chunks +7 lines, -13 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
Sven Panne
9 years, 5 months ago (2011-07-22 09:58:03 UTC) #1
danno
9 years, 5 months ago (2011-07-22 12:16:14 UTC) #2
LGTM

Powered by Google App Engine
This is Rietveld 408576698