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

Issue 553056: Inline smi code in count operations in the full code generator. (Closed)

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

Description

Inline smi code in count operations in the full code generator. We now test for a smi before calling ToNumber and inline the smi increment/decrement for ++ and --. There only a small increase in code size but loops in top-level code are becoming much faster as a result. Committed: http://code.google.com/p/v8/source/detail?r=3681

Patch Set 1 #

Total comments: 2

Patch Set 2 : '' #

Unified diffs Side-by-side diffs Delta from patch set Stats (+83 lines, -10 lines) Patch
M src/arm/full-codegen-arm.cc View 1 3 chunks +24 lines, -4 lines 0 comments Download
M src/ia32/full-codegen-ia32.cc View 4 chunks +29 lines, -3 lines 0 comments Download
M src/x64/full-codegen-x64.cc View 4 chunks +30 lines, -3 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
fschneider
Nothing new here for ia-32 and x64. This changelist includes only the changes to CountOperation ...
10 years, 11 months ago (2010-01-22 13:47:30 UTC) #1
Erik Corry
LGTM http://codereview.chromium.org/553056/diff/1/4 File src/arm/full-codegen-arm.cc (right): http://codereview.chromium.org/553056/diff/1/4#newcode1443 src/arm/full-codegen-arm.cc:1443: __ add(r0, r0, Operand(r1)); Small immediates can be ...
10 years, 11 months ago (2010-01-22 13:56:43 UTC) #2
fschneider
10 years, 11 months ago (2010-01-22 14:07:42 UTC) #3
http://codereview.chromium.org/553056/diff/1/4
File src/arm/full-codegen-arm.cc (right):

http://codereview.chromium.org/553056/diff/1/4#newcode1443
src/arm/full-codegen-arm.cc:1443: __ add(r0, r0, Operand(r1));
On 2010/01/22 13:56:43, Erik Corry wrote:
> Small immediates can be added and substracted in the instruction.  Then we
only
> need to load r1 on the slow path.

Done.

Powered by Google App Engine
This is Rietveld 408576698