Revision: 14823
Author: [email protected]
Date: Mon May 27 05:39:01 2013
Log: MIPS: Remove HCheckSmi, LCheckSmi and rename LCheckSmiAndReturn
to LCheckSmi.
Port r14816 (0485153d)
BUG=
Review URL: https://codereview.chromium.org/16035008
http://code.google.com/p/v8/source/detail?r=14823
Modified:
/branches/bleeding_edge/src/mips/lithium-codegen-mips.cc
/branches/bleeding_edge/src/mips/lithium-mips.cc
/branches/bleeding_edge/src/mips/lithium-mips.h
=======================================
--- /branches/bleeding_edge/src/mips/lithium-codegen-mips.cc Fri May 24
13:20:08 2013
+++ /branches/bleeding_edge/src/mips/lithium-codegen-mips.cc Mon May 27
05:39:01 2013
@@ -4945,13 +4945,6 @@
__ SmiTagCheckOverflow(result_reg, result_reg, scratch1);
DeoptimizeIf(lt, instr->environment(), scratch1, Operand(zero_reg));
}
-
-
-void LCodeGen::DoCheckSmiAndReturn(LCheckSmiAndReturn* instr) {
- LOperand* input = instr->value();
- __ And(at, ToRegister(input), Operand(kSmiTagMask));
- DeoptimizeIf(ne, instr->environment(), at, Operand(zero_reg));
-}
void LCodeGen::DoCheckSmi(LCheckSmi* instr) {
=======================================
--- /branches/bleeding_edge/src/mips/lithium-mips.cc Fri May 24 13:20:08
2013
+++ /branches/bleeding_edge/src/mips/lithium-mips.cc Mon May 27 05:39:01
2013
@@ -1778,8 +1778,7 @@
if (val->type().IsSmi()) {
return DefineSameAsFirst(new(zone()) LDummyUse(value));
}
- return AssignEnvironment(
- DefineSameAsFirst(new(zone()) LCheckSmiAndReturn(value)));
+ return AssignEnvironment(DefineSameAsFirst(new(zone())
LCheckSmi(value)));
} else {
ASSERT(to.IsInteger32());
LOperand* value = NULL;
@@ -1884,12 +1883,6 @@
LCheckPrototypeMaps* result = new(zone()) LCheckPrototypeMaps(temp1,
temp2);
return AssignEnvironment(result);
}
-
-
-LInstruction* LChunkBuilder::DoCheckSmi(HCheckSmi* instr) {
- LOperand* value = UseRegisterAtStart(instr->value());
- return AssignEnvironment(new(zone()) LCheckSmi(value));
-}
LInstruction* LChunkBuilder::DoCheckFunction(HCheckFunction* instr) {
=======================================
--- /branches/bleeding_edge/src/mips/lithium-mips.h Fri May 24 13:20:08 2013
+++ /branches/bleeding_edge/src/mips/lithium-mips.h Mon May 27 05:39:01 2013
@@ -76,7 +76,6 @@
V(CheckNonSmi) \
V(CheckPrototypeMaps) \
V(CheckSmi) \
- V(CheckSmiAndReturn) \
V(ClampDToUint8) \
V(ClampIToUint8) \
V(ClampTToUint8) \
@@ -2325,7 +2324,7 @@
};
-class LCheckSmi: public LTemplateInstruction<0, 1, 0> {
+class LCheckSmi: public LTemplateInstruction<1, 1, 0> {
public:
explicit LCheckSmi(LOperand* value) {
inputs_[0] = value;
@@ -2337,18 +2336,6 @@
};
-class LCheckSmiAndReturn: public LTemplateInstruction<1, 1, 0> {
- public:
- explicit LCheckSmiAndReturn(LOperand* value) {
- inputs_[0] = value;
- }
-
- LOperand* value() { return inputs_[0]; }
-
- DECLARE_CONCRETE_INSTRUCTION(CheckSmiAndReturn, "check-smi-and-return")
-};
-
-
class LCheckNonSmi: public LTemplateInstruction<0, 1, 0> {
public:
explicit LCheckNonSmi(LOperand* value) {
--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
---
You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
For more options, visit https://groups.google.com/groups/opt_out.
