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

Issue 113402: Improve algorithm for detaching and attaching a virtual frame to the code... (Closed)

Created:
11 years, 7 months ago by William Hesse
Modified:
9 years, 7 months ago
CC:
v8-dev
Visibility:
Public.

Description

Improve algorithm for detaching and attaching a virtual frame to the code generator. Inline copying of a register file. Committed: http://code.google.com/p/v8/source/detail?r=1954

Patch Set 1 #

Total comments: 2

Patch Set 2 : '' #

Unified diffs Side-by-side diffs Delta from patch set Stats (+59 lines, -42 lines) Patch
M src/arm/virtual-frame-arm.h View 1 chunk +18 lines, -2 lines 0 comments Download
M src/ia32/virtual-frame-ia32.h View 1 chunk +18 lines, -2 lines 0 comments Download
M src/register-allocator.h View 1 chunk +5 lines, -1 line 0 comments Download
M src/register-allocator.cc View 1 chunk +0 lines, -10 lines 0 comments Download
M src/virtual-frame.cc View 1 chunk +0 lines, -24 lines 0 comments Download
M src/x64/virtual-frame-x64.h View 1 chunk +18 lines, -3 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
William Hesse
11 years, 7 months ago (2009-05-14 15:32:34 UTC) #1
Kevin Millikin (Chromium)
LGTM. http://codereview.chromium.org/113402/diff/1/6 File src/arm/virtual-frame-arm.h (right): http://codereview.chromium.org/113402/diff/1/6#newcode134 Line 134: Register temp = { i }; It ...
11 years, 7 months ago (2009-05-14 15:41:49 UTC) #2
William Hesse
11 years, 7 months ago (2009-05-15 15:44:40 UTC) #3
http://codereview.chromium.org/113402/diff/1/6
File src/arm/virtual-frame-arm.h (right):

http://codereview.chromium.org/113402/diff/1/6#newcode134
Line 134: Register temp = { i };
On 2009/05/14 15:41:49, Kevin Millikin wrote:
> It might be better to overload RegisterAllocator::Unuse to have an int (and
call
> it in the Register version).

This completely disappears in the generated assembly code, so is it worthwhile
adding the extra (inlined) functions just to make this look nicer?

Powered by Google App Engine
This is Rietveld 408576698