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

Unified Diff: runtime/vm/flow_graph_compiler_ia32.cc

Issue 12412013: Add pass to remove empty blocks and recompute fall-through targets. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 7 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: runtime/vm/flow_graph_compiler_ia32.cc
diff --git a/runtime/vm/flow_graph_compiler_ia32.cc b/runtime/vm/flow_graph_compiler_ia32.cc
index 8c02c40e4aca296b613d4e50bdbec525d005e63f..41aa3fbede4cddae5b7b86f63124f551dd93c90f 100644
--- a/runtime/vm/flow_graph_compiler_ia32.cc
+++ b/runtime/vm/flow_graph_compiler_ia32.cc
@@ -32,8 +32,8 @@ FlowGraphCompiler::~FlowGraphCompiler() {
// BlockInfos are zone-allocated, so their destructors are not called.
// Verify the labels explicitly here.
for (int i = 0; i < block_info_.length(); ++i) {
- ASSERT(!block_info_[i]->label.IsLinked());
- ASSERT(!block_info_[i]->label.HasNear());
+ ASSERT(!block_info_[i]->jump_label()->IsLinked());
+ ASSERT(!block_info_[i]->jump_label()->HasNear());
}
}
@@ -1437,7 +1437,7 @@ void FlowGraphCompiler::EmitDoubleCompareBranch(Condition true_condition,
assembler()->comisd(left, right);
BlockEntryInstr* nan_result = (true_condition == NOT_EQUAL) ?
branch->true_successor() : branch->false_successor();
- assembler()->j(PARITY_EVEN, GetBlockLabel(nan_result));
+ assembler()->j(PARITY_EVEN, GetJumpLabel(nan_result));
branch->EmitBranchOnCondition(this, true_condition);
}

Powered by Google App Engine
This is Rietveld 408576698