[vm/checker] Keep private method order consistent with header

aartbik · commit-bot@chromium.org · commit 1b0469184ea9 · 2019-08-08T16:29:41.000Z
Rationale: Use a consistent order for methods in header and CC. Change-Id: Idf887346aea402bae1a9804183f49ef10b2e04c7 Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/112272 Reviewed-by: Samir Jindel <sjindel@google.com> Commit-Queue: Aart Bik <ajcbik@google.com>
diff --git a/runtime/vm/compiler/backend/flow_graph_checker.cc b/runtime/vm/compiler/backend/flow_graph_checker.cc
@@ -349,23 +349,6 @@ void FlowGraphChecker::VisitConstant(ConstantInstr* constant) {
   // ASSERT(constant->GetBlock() == flow_graph_->graph_entry());
 }
 
-void FlowGraphChecker::VisitInstanceCall(InstanceCallInstr* instr) {
-  const Function& function = flow_graph_->function();
-
-  // Force-optimized functions may not have instance calls inside them because
-  // we do not reset ICData for these.
-  ASSERT(!function.ForceOptimize());
-}
-
-void FlowGraphChecker::VisitPolymorphicInstanceCall(
-    PolymorphicInstanceCallInstr* instr) {
-  const Function& function = flow_graph_->function();
-
-  // Force-optimized functions may not have instance calls inside them because
-  // we do not reset ICData for these.
-  ASSERT(!function.ForceOptimize());
-}
-
 void FlowGraphChecker::VisitPhi(PhiInstr* phi) {
   // Make sure the definition of each input value of a Phi dominates
   // the corresponding incoming edge, as defined by order.
@@ -393,6 +376,19 @@ void FlowGraphChecker::VisitRedefinition(RedefinitionInstr* def) {
   ASSERT(def->value()->definition() != def);
 }
 
+void FlowGraphChecker::VisitInstanceCall(InstanceCallInstr* instr) {
+  // Force-optimized functions may not have instance calls inside them because
+  // we do not reset ICData for these.
+  ASSERT(!flow_graph_->function().ForceOptimize());
+}
+
+void FlowGraphChecker::VisitPolymorphicInstanceCall(
+    PolymorphicInstanceCallInstr* instr) {
+  // Force-optimized functions may not have instance calls inside them because
+  // we do not reset ICData for these.
+  ASSERT(!flow_graph_->function().ForceOptimize());
+}
+
 // Main entry point of graph checker.
 void FlowGraphChecker::Check(const char* pass_name) {
   if (FLAG_trace_compiler) {
