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

Issue 594008: Initial implementation of fast path operation for bitwise OR. (Closed)

Created:
10 years, 10 months ago by Kevin Millikin (Chromium)
Modified:
9 years, 7 months ago
CC:
v8-dev
Visibility:
Public.

Description

Initial implementation of fast path operation for bitwise OR. Support a binary operation (bitwise OR) so long as it's not nested in the left subexpression. This ensures that the expression stack never has height greater than two and so can be kept fully in registers. The bounded expression stack height and the absence of any side effects on the fast path allows us to still bailout out to the very beginning of the function if any of our fast-path checks fail. Committed: http://code.google.com/p/v8/source/detail?r=3822

Patch Set 1 #

Total comments: 4

Patch Set 2 : Fixed bug, added test. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+448 lines, -71 lines) Patch
M src/arm/fast-codegen-arm.cc View 1 2 chunks +57 lines, -4 lines 0 comments Download
M src/ast.h View 10 chunks +25 lines, -0 lines 0 comments Download
M src/data-flow.cc View 2 chunks +11 lines, -10 lines 0 comments Download
M src/fast-codegen.h View 3 chunks +28 lines, -18 lines 0 comments Download
M src/fast-codegen.cc View 8 chunks +123 lines, -16 lines 0 comments Download
M src/ia32/fast-codegen-ia32.cc View 1 5 chunks +73 lines, -12 lines 0 comments Download
M src/x64/fast-codegen-x64.cc View 1 3 chunks +67 lines, -11 lines 0 comments Download
A test/mjsunit/compiler/this-property-refs.js View 1 chunk +64 lines, -0 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
Kevin Millikin (Chromium)
I'll put together some dedicated tests as a separate change.
10 years, 10 months ago (2010-02-09 12:27:43 UTC) #1
Mads Ager (chromium)
LGTM except for the inobject property load case. http://codereview.chromium.org/594008/diff/1/2 File src/arm/fast-codegen-arm.cc (right): http://codereview.chromium.org/594008/diff/1/2#newcode111 src/arm/fast-codegen-arm.cc:111: __ ...
10 years, 10 months ago (2010-02-09 13:15:44 UTC) #2
Kevin Millikin (Chromium)
10 years, 10 months ago (2010-02-09 13:26:07 UTC) #3
http://codereview.chromium.org/594008/diff/1/2
File src/arm/fast-codegen-arm.cc (right):

http://codereview.chromium.org/594008/diff/1/2#newcode111
src/arm/fast-codegen-arm.cc:111: __ ldr(destination(),
FieldMemOperand(scratch0(), offset));
On 2010/02/09 13:15:44, Mads Ager wrote:
> scratch0() receiver_reg()?

Yes, thank you.

Powered by Google App Engine
This is Rietveld 408576698