Revision: 3888
Author: [email protected]
Date: Wed Feb 17 05:16:53 2010
Log: Remove obsolete bug TODO
Review URL: http://codereview.chromium.org/618006
http://code.google.com/p/v8/source/detail?r=3888
Modified:
/branches/bleeding_edge/src/arm/codegen-arm.cc
/branches/bleeding_edge/src/ia32/codegen-ia32.cc
/branches/bleeding_edge/src/ia32/virtual-frame-ia32.cc
/branches/bleeding_edge/src/x64/codegen-x64.cc
/branches/bleeding_edge/src/x64/virtual-frame-x64.cc
=======================================
--- /branches/bleeding_edge/src/arm/codegen-arm.cc Wed Feb 17 00:26:50 2010
+++ /branches/bleeding_edge/src/arm/codegen-arm.cc Wed Feb 17 05:16:53 2010
@@ -4411,7 +4411,6 @@
// Call IC code.
Handle<Code>
ic(Builtins::builtin(Builtins::KeyedStoreIC_Initialize));
- // TODO(1222589): Make the IC grab the values from the stack.
frame->EmitPop(r0); // value
frame->CallCodeObject(ic, RelocInfo::CODE_TARGET, 0);
frame->EmitPush(r0);
=======================================
--- /branches/bleeding_edge/src/ia32/codegen-ia32.cc Wed Feb 17 00:26:50
2010
+++ /branches/bleeding_edge/src/ia32/codegen-ia32.cc Wed Feb 17 05:16:53
2010
@@ -5790,7 +5790,6 @@
switch (op) {
case Token::SUB: {
GenericUnaryOpStub stub(Token::SUB, overwrite);
- // TODO(1222589): remove dependency of TOS being cached inside stub
Result operand = frame_->Pop();
Result answer = frame_->CallStub(&stub, &operand);
frame_->Push(&answer);
=======================================
--- /branches/bleeding_edge/src/ia32/virtual-frame-ia32.cc Mon Feb 15
06:24:38 2010
+++ /branches/bleeding_edge/src/ia32/virtual-frame-ia32.cc Wed Feb 17
05:16:53 2010
@@ -982,7 +982,6 @@
// expects value in eax and key and receiver on the stack. It does
// not drop the key and receiver.
Handle<Code> ic(Builtins::builtin(Builtins::KeyedStoreIC_Initialize));
- // TODO(1222589): Make the IC grab the values from the stack.
Result value = Pop();
PrepareForCall(2, 0); // Two stack args, neither callee-dropped.
value.ToRegister(eax);
=======================================
--- /branches/bleeding_edge/src/x64/codegen-x64.cc Wed Feb 17 00:26:50 2010
+++ /branches/bleeding_edge/src/x64/codegen-x64.cc Wed Feb 17 05:16:53 2010
@@ -3071,7 +3071,6 @@
case Token::SUB: {
GenericUnaryOpStub stub(Token::SUB, overwrite);
- // TODO(1222589): remove dependency of TOS being cached inside stub
Result operand = frame_->Pop();
Result answer = frame_->CallStub(&stub, &operand);
frame_->Push(&answer);
=======================================
--- /branches/bleeding_edge/src/x64/virtual-frame-x64.cc Mon Feb 15
06:24:38 2010
+++ /branches/bleeding_edge/src/x64/virtual-frame-x64.cc Wed Feb 17
05:16:53 2010
@@ -1033,7 +1033,6 @@
// expects value in rax and key and receiver on the stack. It does
// not drop the key and receiver.
Handle<Code> ic(Builtins::builtin(Builtins::KeyedStoreIC_Initialize));
- // TODO(1222589): Make the IC grab the values from the stack.
Result value = Pop();
PrepareForCall(2, 0); // Two stack args, neither callee-dropped.
value.ToRegister(rax);
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
