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

Unified Diff: runtime/vm/branch_optimizer.cc

Issue 2974233002: VM: Re-format to use at most one newline between functions (Closed)
Patch Set: Rebase and merge Created 3 years, 5 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
« no previous file with comments | « runtime/vm/branch_optimizer.h ('k') | runtime/vm/cha.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: runtime/vm/branch_optimizer.cc
diff --git a/runtime/vm/branch_optimizer.cc b/runtime/vm/branch_optimizer.cc
index 0048b7ed637da3a17cc9cff4b85cb046595f6954..3693eda40191dfeae53141cd63b17a070defea2c 100644
--- a/runtime/vm/branch_optimizer.cc
+++ b/runtime/vm/branch_optimizer.cc
@@ -29,7 +29,6 @@ static bool PhiHasSingleUse(PhiInstr* phi, Value* use) {
return true;
}
-
bool BranchSimplifier::Match(JoinEntryInstr* block) {
// Match the pattern of a branch on a comparison whose left operand is a
// phi from the same block, and whose right operand is a constant.
@@ -59,7 +58,6 @@ bool BranchSimplifier::Match(JoinEntryInstr* block) {
(block->phis()->length() == 1);
}
-
JoinEntryInstr* BranchSimplifier::ToJoinEntry(Zone* zone,
TargetEntryInstr* target) {
// Convert a target block into a join block. Branches will be duplicated
@@ -74,7 +72,6 @@ JoinEntryInstr* BranchSimplifier::ToJoinEntry(Zone* zone,
return join;
}
-
BranchInstr* BranchSimplifier::CloneBranch(Zone* zone,
BranchInstr* branch,
Value* new_left,
@@ -87,7 +84,6 @@ BranchInstr* BranchSimplifier::CloneBranch(Zone* zone,
return new_branch;
}
-
void BranchSimplifier::Simplify(FlowGraph* flow_graph) {
// Optimize some branches that test the value of a phi. When it is safe
// to do so, push the branch to each of the predecessor blocks. This is
@@ -223,7 +219,6 @@ void BranchSimplifier::Simplify(FlowGraph* flow_graph) {
}
}
-
static bool IsTrivialBlock(BlockEntryInstr* block, Definition* defn) {
return (block->IsTargetEntry() && (block->PredecessorCount() == 1)) &&
((block->next() == block->last_instruction()) ||
@@ -231,7 +226,6 @@ static bool IsTrivialBlock(BlockEntryInstr* block, Definition* defn) {
(defn->next() == block->last_instruction())));
}
-
static void EliminateTrivialBlock(BlockEntryInstr* block,
Definition* instr,
IfThenElseInstr* before) {
@@ -245,7 +239,6 @@ static void EliminateTrivialBlock(BlockEntryInstr* block,
}
}
-
void IfConverter::Simplify(FlowGraph* flow_graph) {
Zone* zone = flow_graph->zone();
bool changed = false;
@@ -346,5 +339,4 @@ void IfConverter::Simplify(FlowGraph* flow_graph) {
}
}
-
} // namespace dart
« no previous file with comments | « runtime/vm/branch_optimizer.h ('k') | runtime/vm/cha.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698