diff options
author | Calin Juravle <calin@google.com> | 2014-06-24 10:57:35 +0000 |
---|---|---|
committer | Gerrit Code Review <noreply-gerritcodereview@google.com> | 2014-06-24 06:22:45 +0000 |
commit | 1bc67722c1bcf26934900768a60ca0bfb0a6d1a0 (patch) | |
tree | 68aed0224286c6e668d653b13a41bb767655472e /compiler | |
parent | 7db22606228d21b3adba04851409a4a78e6b4263 (diff) | |
parent | 08f7a2d06b915a5e21ded648d9feee519afd2f76 (diff) | |
download | art-1bc67722c1bcf26934900768a60ca0bfb0a6d1a0.zip art-1bc67722c1bcf26934900768a60ca0bfb0a6d1a0.tar.gz art-1bc67722c1bcf26934900768a60ca0bfb0a6d1a0.tar.bz2 |
Merge "Log profile info only on debug builds."
Diffstat (limited to 'compiler')
-rw-r--r-- | compiler/driver/compiler_driver.cc | 21 |
1 files changed, 13 insertions, 8 deletions
diff --git a/compiler/driver/compiler_driver.cc b/compiler/driver/compiler_driver.cc index 3e326f0..4a331fc 100644 --- a/compiler/driver/compiler_driver.cc +++ b/compiler/driver/compiler_driver.cc @@ -2054,7 +2054,9 @@ bool CompilerDriver::SkipCompilation(const std::string& method_name) { ProfileFile::ProfileData data; if (!profile_file_.GetProfileData(&data, method_name)) { // Not in profile, no information can be determined. - VLOG(compiler) << "not compiling " << method_name << " because it's not in the profile"; + if (kIsDebugBuild) { + VLOG(compiler) << "not compiling " << method_name << " because it's not in the profile"; + } return true; } @@ -2063,13 +2065,16 @@ bool CompilerDriver::SkipCompilation(const std::string& method_name) { // falls inside a bucket. bool compile = data.GetTopKUsedPercentage() - data.GetUsedPercent() <= compiler_options_->GetTopKProfileThreshold(); - if (compile) { - LOG(INFO) << "compiling method " << method_name << " because its usage is part of top " - << data.GetTopKUsedPercentage() << "% with a percent of " << data.GetUsedPercent() << "%" - << " (topKThreshold=" << compiler_options_->GetTopKProfileThreshold() << ")"; - } else { - VLOG(compiler) << "not compiling method " << method_name << " because it's not part of leading " - << compiler_options_->GetTopKProfileThreshold() << "% samples)"; + if (kIsDebugBuild) { + if (compile) { + LOG(INFO) << "compiling method " << method_name << " because its usage is part of top " + << data.GetTopKUsedPercentage() << "% with a percent of " << data.GetUsedPercent() << "%" + << " (topKThreshold=" << compiler_options_->GetTopKProfileThreshold() << ")"; + } else { + VLOG(compiler) << "not compiling method " << method_name + << " because it's not part of leading " << compiler_options_->GetTopKProfileThreshold() + << "% samples)"; + } } return !compile; } |