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

Issue 101016: Improve register allocation of left shift operation. Add tests... (Closed)

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

Description

Improve register allocation of left shift operation. Add tests for all shift operations. Committed: http://code.google.com/p/v8/source/detail?r=1825

Patch Set 1 #

Patch Set 2 : '' #

Total comments: 1

Patch Set 3 : '' #

Unified diffs Side-by-side diffs Delta from patch set Stats (+410 lines, -19 lines) Patch
M src/ia32/codegen-ia32.cc View 1 2 4 chunks +19 lines, -18 lines 0 comments Download
M test/mjsunit/smi-ops.js View 1 2 2 chunks +391 lines, -1 line 0 comments Download

Messages

Total messages: 2 (0 generated)
William Hesse
Resolves TODO(207) issue 207.
11 years, 7 months ago (2009-04-29 11:53:36 UTC) #1
Kevin Millikin (Chromium)
11 years, 7 months ago (2009-04-29 14:17:27 UTC) #2
LGTM.  I have to confess that I didn't read every line of the tests.

http://codereview.chromium.org/101016/diff/2001/1002
File src/ia32/codegen-ia32.cc (right):

http://codereview.chromium.org/101016/diff/2001/1002#newcode5908
Line 5908: *left = answer;
Indentation?

Powered by Google App Engine
This is Rietveld 408576698