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

Unified Diff: runtime/vm/flow_graph_compiler_x64.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_x64.cc
diff --git a/runtime/vm/flow_graph_compiler_x64.cc b/runtime/vm/flow_graph_compiler_x64.cc
index d5d6c696d23b8a26c2141b7851e44996cbb803c1..079f3d19d31b2f5cc2750b893c543e4205b306ec 100644
--- a/runtime/vm/flow_graph_compiler_x64.cc
+++ b/runtime/vm/flow_graph_compiler_x64.cc
@@ -31,8 +31,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