Revision: 6163
Author: [email protected]
Date: Tue Jan 4 06:35:01 2011
Log: Remove unused policy from register allocator.
DefineSameAsAny is never used and currently does exactly the
same as DefineSameAsFirst.
Review URL: http://codereview.chromium.org/6025014
http://code.google.com/p/v8/source/detail?r=6163
Modified:
/branches/bleeding_edge/src/arm/lithium-arm.cc
/branches/bleeding_edge/src/arm/lithium-arm.h
/branches/bleeding_edge/src/ia32/lithium-ia32.cc
/branches/bleeding_edge/src/ia32/lithium-ia32.h
/branches/bleeding_edge/src/lithium-allocator.cc
/branches/bleeding_edge/src/lithium-allocator.h
=======================================
--- /branches/bleeding_edge/src/arm/lithium-arm.cc Tue Jan 4 06:32:54 2011
+++ /branches/bleeding_edge/src/arm/lithium-arm.cc Tue Jan 4 06:35:01 2011
@@ -765,11 +765,6 @@
LInstruction* LChunkBuilder::DefineAsSpilled(LInstruction* instr, int
index) {
return Define(instr, new LUnallocated(LUnallocated::FIXED_SLOT, index));
}
-
-
-LInstruction* LChunkBuilder::DefineSameAsAny(LInstruction* instr) {
- return Define(instr, new LUnallocated(LUnallocated::SAME_AS_ANY_INPUT));
-}
LInstruction* LChunkBuilder::DefineSameAsFirst(LInstruction* instr) {
=======================================
--- /branches/bleeding_edge/src/arm/lithium-arm.h Tue Jan 4 06:32:54 2011
+++ /branches/bleeding_edge/src/arm/lithium-arm.h Tue Jan 4 06:35:01 2011
@@ -2061,7 +2061,6 @@
LInstruction* Define(LInstruction* instr);
LInstruction* DefineAsRegister(LInstruction* instr);
LInstruction* DefineAsSpilled(LInstruction* instr, int index);
- LInstruction* DefineSameAsAny(LInstruction* instr);
LInstruction* DefineSameAsFirst(LInstruction* instr);
LInstruction* DefineFixed(LInstruction* instr, Register reg);
LInstruction* DefineFixedDouble(LInstruction* instr, DoubleRegister reg);
=======================================
--- /branches/bleeding_edge/src/ia32/lithium-ia32.cc Tue Jan 4 03:36:20
2011
+++ /branches/bleeding_edge/src/ia32/lithium-ia32.cc Tue Jan 4 06:35:01
2011
@@ -765,11 +765,6 @@
LInstruction* LChunkBuilder::DefineAsSpilled(LInstruction* instr, int
index) {
return Define(instr, new LUnallocated(LUnallocated::FIXED_SLOT, index));
}
-
-
-LInstruction* LChunkBuilder::DefineSameAsAny(LInstruction* instr) {
- return Define(instr, new LUnallocated(LUnallocated::SAME_AS_ANY_INPUT));
-}
LInstruction* LChunkBuilder::DefineSameAsFirst(LInstruction* instr) {
=======================================
--- /branches/bleeding_edge/src/ia32/lithium-ia32.h Thu Dec 30 11:30:42 2010
+++ /branches/bleeding_edge/src/ia32/lithium-ia32.h Tue Jan 4 06:35:01 2011
@@ -2087,7 +2087,6 @@
LInstruction* Define(LInstruction* instr);
LInstruction* DefineAsRegister(LInstruction* instr);
LInstruction* DefineAsSpilled(LInstruction* instr, int index);
- LInstruction* DefineSameAsAny(LInstruction* instr);
LInstruction* DefineSameAsFirst(LInstruction* instr);
LInstruction* DefineFixed(LInstruction* instr, Register reg);
LInstruction* DefineFixedDouble(LInstruction* instr, XMMRegister reg);
=======================================
--- /branches/bleeding_edge/src/lithium-allocator.cc Tue Jan 4 03:36:20
2011
+++ /branches/bleeding_edge/src/lithium-allocator.cc Tue Jan 4 06:35:01
2011
@@ -106,9 +106,6 @@
case LUnallocated::SAME_AS_FIRST_INPUT:
stream->Add("(1)");
break;
- case LUnallocated::SAME_AS_ANY_INPUT:
- stream->Add("(A)");
- break;
case LUnallocated::ANY:
stream->Add("(-)");
break;
=======================================
--- /branches/bleeding_edge/src/lithium-allocator.h Mon Jan 3 09:02:15 2011
+++ /branches/bleeding_edge/src/lithium-allocator.h Tue Jan 4 06:35:01 2011
@@ -205,7 +205,6 @@
MUST_HAVE_REGISTER,
WRITABLE_REGISTER,
SAME_AS_FIRST_INPUT,
- SAME_AS_ANY_INPUT,
IGNORE
};
@@ -276,7 +275,7 @@
return policy() == WRITABLE_REGISTER || policy() == MUST_HAVE_REGISTER;
}
bool HasSameAsInputPolicy() const {
- return policy() == SAME_AS_FIRST_INPUT || policy() ==
SAME_AS_ANY_INPUT;
+ return policy() == SAME_AS_FIRST_INPUT;
}
Policy policy() const { return PolicyField::decode(value_); }
void set_policy(Policy policy) {
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
