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

Issue 6877036: Make predicates on lithium instruction classes non-virtual. (Closed)

Created:
9 years, 8 months ago by fschneider
Modified:
9 years, 6 months ago
CC:
v8-dev, Satish
Visibility:
Public.

Description

Make predicates on lithium instruction classes non-virtual. This is the lithium analog to the refactoring of the HIR classes in r7659. Committed: http://code.google.com/p/v8/source/detail?r=7662

Patch Set 1 #

Patch Set 2 : Fix lintos #

Total comments: 2
Unified diffs Side-by-side diffs Delta from patch set Stats (+69 lines, -67 lines) Patch
M src/arm/lithium-arm.h View 1 7 chunks +23 lines, -22 lines 2 comments Download
M src/ia32/lithium-ia32.h View 1 7 chunks +23 lines, -23 lines 0 comments Download
M src/x64/lithium-x64.h View 1 7 chunks +23 lines, -22 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
fschneider
9 years, 8 months ago (2011-04-19 11:05:40 UTC) #1
fschneider
This is a similar refactoring as the one I did for the hydrogen instruction classes.
9 years, 8 months ago (2011-04-19 11:06:38 UTC) #2
Kevin Millikin (Chromium)
LGTM. http://codereview.chromium.org/6877036/diff/3002/src/arm/lithium-arm.h File src/arm/lithium-arm.h (right): http://codereview.chromium.org/6877036/diff/3002/src/arm/lithium-arm.h#newcode169 src/arm/lithium-arm.h:169: virtual Opcode opcode() const { return LInstruction::k##type; } ...
9 years, 8 months ago (2011-04-19 12:20:30 UTC) #3
fschneider
9 years, 8 months ago (2011-04-19 12:31:40 UTC) #4
On 2011/04/19 12:20:30, Kevin Millikin wrote:
> LGTM.
> 
> http://codereview.chromium.org/6877036/diff/3002/src/arm/lithium-arm.h
> File src/arm/lithium-arm.h (right):
> 
>
http://codereview.chromium.org/6877036/diff/3002/src/arm/lithium-arm.h#newcod...
> src/arm/lithium-arm.h:169: virtual Opcode opcode() const { return
> LInstruction::k##type; } \
> Thank you.
> 
>
http://codereview.chromium.org/6877036/diff/3002/src/arm/lithium-arm.h#newcod...
> src/arm/lithium-arm.h:1109: virtual Opcode opcode() const { return
> LInstruction::kArithmeticD; }
> Maybe add a comment here and below that DECLARE_CONCRETE_INSTRUCTION doesn't
> work because the mnemonic depends on the op?

I'll replace these aggregate instructions pretty soon. They don't seem to buy us
much.

Powered by Google App Engine
This is Rietveld 408576698