From d92b37fa9c9c3dbff9d2ee2e02b4f97e68e256cb Mon Sep 17 00:00:00 2001 From: Stefano Cianciulli Date: Tue, 8 Nov 2022 12:03:50 +0000 Subject: Report value metrics as increments / deltas This CL changes value metrics to be represented as increment / deltas rather than absolute values, which will be aggregated on device by summing them instead of taking the maximum value produced. This should make the value metrics representative of all processes across restarts, rather than reporting only the process producing the maximum value (for which numerator and denominator could have been produced by different processes). Ignore-AOSP-First: this topic includes changes in the atoms.proto file, for which AOSP and internal repo have diverged, causing merge conflicts while running presubmits. Bug: 246330418 Test: atest ArtGtestsTargetChroot (in particular art_standalone_libartbase_tests) Change-Id: I9b6e67143fbbf4e93748746491ac23b02543cd28 --- compiler/jit/jit_compiler.cc | 2 ++ 1 file changed, 2 insertions(+) (limited to 'compiler/jit/jit_compiler.cc') diff --git a/compiler/jit/jit_compiler.cc b/compiler/jit/jit_compiler.cc index d2f5aa6799..e67236769e 100644 --- a/compiler/jit/jit_compiler.cc +++ b/compiler/jit/jit_compiler.cc @@ -203,6 +203,8 @@ bool JitCompiler::CompileMethod( VLOG(jit) << "Compilation of " << method->PrettyMethod() << " took " << PrettyDuration(UsToNs(duration_us)); runtime->GetMetrics()->JitMethodCompileCount()->AddOne(); + runtime->GetMetrics()->JitMethodCompileTotalTimeDelta()->Add(duration_us); + runtime->GetMetrics()->JitMethodCompileCountDelta()->AddOne(); } // Trim maps to reduce memory usage. -- cgit v1.2.3-59-g8ed1b