From 69ba7b7112c2277ac225615b37e6df74c055740d Mon Sep 17 00:00:00 2001 From: David Brazdil Date: Tue, 23 Jun 2015 18:27:30 +0100 Subject: ART: Run GraphChecker after Builder and SsaBuilder This patch refactors the way GraphChecker is invoked, utilizing the same scoping mechanism as pass timing and graph visualizer. Therefore, GraphChecker will now run not just after instances of HOptimization but after the builders and reg alloc, too. Change-Id: I8173b98b79afa95e1fcbf3ac9630a873d7f6c1d4 --- compiler/optimizing/optimization.cc | 23 ----------------------- 1 file changed, 23 deletions(-) (limited to 'compiler/optimizing/optimization.cc') diff --git a/compiler/optimizing/optimization.cc b/compiler/optimizing/optimization.cc index c46a21955c..3d769491a5 100644 --- a/compiler/optimizing/optimization.cc +++ b/compiler/optimizing/optimization.cc @@ -16,9 +16,6 @@ #include "optimization.h" -#include "base/dumpable.h" -#include "graph_checker.h" - namespace art { void HOptimization::MaybeRecordStat(MethodCompilationStat compilation_stat, size_t count) const { @@ -27,24 +24,4 @@ void HOptimization::MaybeRecordStat(MethodCompilationStat compilation_stat, size } } -void HOptimization::Check() { - if (kIsDebugBuild) { - if (is_in_ssa_form_) { - SSAChecker checker(graph_->GetArena(), graph_); - checker.Run(); - if (!checker.IsValid()) { - LOG(FATAL) << "Error after " << GetPassName() << ": " - << Dumpable(checker); - } - } else { - GraphChecker checker(graph_->GetArena(), graph_); - checker.Run(); - if (!checker.IsValid()) { - LOG(FATAL) << "Error after " << GetPassName() << ": " - << Dumpable(checker); - } - } - } -} - } // namespace art -- cgit v1.2.3-59-g8ed1b