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

Issue 6364007: ARM: Implement DoInstanceOfAndBranch in the lithium code generator (Closed)

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

Description

ARM: Implement DoInstanceOfAndBranch in the lithium code generator. BUG=none TEST=none

Patch Set 1 #

Total comments: 2

Patch Set 2 : Add fixed input register allocation. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+12 lines, -3 lines) Patch
M src/arm/lithium-arm.cc View 1 1 chunk +2 lines, -2 lines 0 comments Download
M src/arm/lithium-codegen-arm.cc View 1 1 chunk +10 lines, -1 line 0 comments Download

Messages

Total messages: 4 (0 generated)
Karl Klose
9 years, 11 months ago (2011-01-25 22:28:59 UTC) #1
Søren Thygesen Gjesse
http://codereview.chromium.org/6364007/diff/1/src/arm/lithium-codegen-arm.cc File src/arm/lithium-codegen-arm.cc (right): http://codereview.chromium.org/6364007/diff/1/src/arm/lithium-codegen-arm.cc#newcode2012 src/arm/lithium-codegen-arm.cc:2012: InstanceofStub stub(InstanceofStub::kArgsInRegisters); Don't you need to change the creation ...
9 years, 11 months ago (2011-01-26 07:38:50 UTC) #2
Karl Klose
http://codereview.chromium.org/6364007/diff/1/src/arm/lithium-codegen-arm.cc File src/arm/lithium-codegen-arm.cc (right): http://codereview.chromium.org/6364007/diff/1/src/arm/lithium-codegen-arm.cc#newcode2012 src/arm/lithium-codegen-arm.cc:2012: InstanceofStub stub(InstanceofStub::kArgsInRegisters); Yes, I fixed it. On 2011/01/26 07:38:51, ...
9 years, 11 months ago (2011-01-26 20:39:18 UTC) #3
Søren Thygesen Gjesse
9 years, 11 months ago (2011-01-26 20:58:22 UTC) #4
LGTM

Powered by Google App Engine
This is Rietveld 408576698