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

Issue 3203005: Start using the overwrite mode from the full codegens to generate... (Closed)

Created:
10 years, 4 months ago by Kasper Lund
Modified:
9 years, 7 months ago
CC:
v8-dev
Visibility:
Public.

Description

Start using the overwrite mode from the full codegens to generate slightly better code and allow passing arguments to binary op stubs in registers on the platforms that support it. Committed: http://code.google.com/p/v8/source/detail?r=5339

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+151 lines, -134 lines) Patch
M src/arm/codegen-arm.cc View 5 chunks +6 lines, -18 lines 0 comments Download
M src/arm/full-codegen-arm.cc View 7 chunks +12 lines, -12 lines 0 comments Download
M src/ast.h View 3 chunks +6 lines, -26 lines 0 comments Download
M src/ast.cc View 1 chunk +36 lines, -0 lines 0 comments Download
M src/full-codegen.h View 2 chunks +9 lines, -1 line 0 comments Download
M src/full-codegen.cc View 3 chunks +16 lines, -1 line 0 comments Download
M src/ia32/code-stubs-ia32.h View 2 chunks +5 lines, -4 lines 0 comments Download
M src/ia32/codegen-ia32.cc View 5 chunks +6 lines, -16 lines 0 comments Download
M src/ia32/full-codegen-ia32.cc View 8 chunks +25 lines, -20 lines 0 comments Download
M src/x64/code-stubs-x64.h View 2 chunks +5 lines, -4 lines 0 comments Download
M src/x64/codegen-x64.cc View 5 chunks +6 lines, -16 lines 0 comments Download
M src/x64/full-codegen-x64.cc View 7 chunks +19 lines, -16 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
Kasper Lund
10 years, 4 months ago (2010-08-25 10:41:07 UTC) #1
fschneider
10 years, 4 months ago (2010-08-25 11:07:32 UTC) #2
LGTM.

Powered by Google App Engine
This is Rietveld 408576698