diff options
| author | Nicolas Geoffray <ngeoffray@google.com> | 2014-02-10 11:59:41 +0000 |
|---|---|---|
| committer | Nicolas Geoffray <ngeoffray@google.com> | 2014-02-11 10:12:55 +0000 |
| commit | ea3fa0b4ba13d7bd7f7c1cd85202ccbe141a35ae (patch) | |
| tree | 7fe7cc462617ff41bf26dc551f2fdc5f25146eda /compiler/oat_test.cc | |
| parent | a52214230ad091359956ed7566670963aedf2045 (diff) | |
| download | art-ea3fa0b4ba13d7bd7f7c1cd85202ccbe141a35ae.zip art-ea3fa0b4ba13d7bd7f7c1cd85202ccbe141a35ae.tar.gz art-ea3fa0b4ba13d7bd7f7c1cd85202ccbe141a35ae.tar.bz2 | |
Re-apply: Implement cumulative timings for CompilerDriver.
The bug was due to how the test infrastructure works:
a compiler driver surives the stack where it is allocated.
Change-Id: I345fe0e4afb2bd15937233db8afb350f09429558
Diffstat (limited to 'compiler/oat_test.cc')
| -rw-r--r-- | compiler/oat_test.cc | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/compiler/oat_test.cc b/compiler/oat_test.cc index b3070b6..f6b511c 100644 --- a/compiler/oat_test.cc +++ b/compiler/oat_test.cc @@ -91,10 +91,12 @@ TEST_F(OatTest, WriteRead) { verification_results_.reset(new VerificationResults); method_inliner_map_.reset(compiler_backend == kQuick ? new DexFileToMethodInlinerMap : nullptr); callbacks_.Reset(verification_results_.get(), method_inliner_map_.get()); + timer_.reset(new CumulativeLogger("Compilation times")); compiler_driver_.reset(new CompilerDriver(verification_results_.get(), method_inliner_map_.get(), compiler_backend, insn_set, - insn_features, false, NULL, 2, true)); + insn_features, false, NULL, 2, true, true, + timer_.get())); jobject class_loader = NULL; if (kCompile) { TimingLogger timings("OatTest::WriteRead", false, false); |