diff options
| author | Mathieu Chartier <mathieuc@google.com> | 2017-07-19 16:55:04 -0700 |
|---|---|---|
| committer | Mathieu Chartier <mathieuc@google.com> | 2017-07-20 09:36:26 -0700 |
| commit | e01b6f674a865a3aef7d66cd91a2d2e226587b50 (patch) | |
| tree | 1d99c6889746d829a86b375b3d88552f29461b7e /compiler/common_compiler_test.cc | |
| parent | e8f48da635c4d07bbe431e5819da8e1fad91a8ef (diff) | |
Move IsVeryLarge check before we create verification results
Otherwise we can end up creating verification results when they are
not actually needed (if the compiler filter was >= quicken).
Bug: 63467744
Test: test-art-host
Change-Id: Ied2a12e0e4c2010f3f660e278c3a5111545ba251
Diffstat (limited to 'compiler/common_compiler_test.cc')
| -rw-r--r-- | compiler/common_compiler_test.cc | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/compiler/common_compiler_test.cc b/compiler/common_compiler_test.cc index 3683695a1b..120389803a 100644 --- a/compiler/common_compiler_test.cc +++ b/compiler/common_compiler_test.cc @@ -207,8 +207,10 @@ void CommonCompilerTest::SetUpRuntimeOptions(RuntimeOptions* options) { compiler_options_.reset(new CompilerOptions); verification_results_.reset(new VerificationResults(compiler_options_.get())); - callbacks_.reset(new QuickCompilerCallbacks(verification_results_.get(), - CompilerCallbacks::CallbackMode::kCompileApp)); + QuickCompilerCallbacks* callbacks = + new QuickCompilerCallbacks(CompilerCallbacks::CallbackMode::kCompileApp); + callbacks->SetVerificationResults(verification_results_.get()); + callbacks_.reset(callbacks); } Compiler::Kind CommonCompilerTest::GetCompilerKind() const { |