Reviewers: jarin,
Description:
Remove testing artifacts from Schedule::AddFoo.
[email protected]
Please review this at https://codereview.chromium.org/641553003/
Base URL: https://v8.googlecode.com/svn/branches/bleeding_edge
Affected files (+1, -11 lines):
M src/compiler/schedule.cc
Index: src/compiler/schedule.cc
diff --git a/src/compiler/schedule.cc b/src/compiler/schedule.cc
index
723415aecf15c560e513fa57c95035491f5ebfaa..0d6e54cd5b6e0e4e34dd502e73dfb6e861c135a7
100644
--- a/src/compiler/schedule.cc
+++ b/src/compiler/schedule.cc
@@ -194,10 +194,6 @@ void Schedule::AddBranch(BasicBlock* block, Node*
branch, BasicBlock* tblock,
AddSuccessor(block, tblock);
AddSuccessor(block, fblock);
SetControlInput(block, branch);
- if (branch->opcode() == IrOpcode::kBranch) {
- // TODO(titzer): require a Branch node here. (sloppy tests).
- SetBlockForNode(block, branch);
- }
}
@@ -205,13 +201,7 @@ void Schedule::AddReturn(BasicBlock* block, Node*
input) {
DCHECK(block->control() == BasicBlock::kNone);
block->set_control(BasicBlock::kReturn);
SetControlInput(block, input);
- if (block != end()) {
- AddSuccessor(block, end());
- }
- if (input->opcode() == IrOpcode::kReturn) {
- // TODO(titzer): require a Return node here. (sloppy tests).
- SetBlockForNode(block, input);
- }
+ if (block != end()) AddSuccessor(block, end());
}
--
--
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/d/optout.
