Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(911)

Unified Diff: src/compilation-statistics.cc

Issue 1331553002: Make --turbo-stats output more self-explanatory. (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: Created 5 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/compilation-statistics.cc
diff --git a/src/compilation-statistics.cc b/src/compilation-statistics.cc
index 2686ff74edb28d646c23dc03db4fe9220ae717f5..ed568cba3fd8650d13deecd94dc4581f1abd78ca 100644
--- a/src/compilation-statistics.cc
+++ b/src/compilation-statistics.cc
@@ -67,16 +67,15 @@ static void WriteLine(std::ostream& os, const char* name,
static_cast<double>(stats.total_allocated_bytes_ * 100) /
static_cast<double>(total_stats.total_allocated_bytes_);
base::OS::SNPrintF(buffer, kBufferSize,
- "%28s %10.3f ms / %5.1f %%"
- "%10u total / %5.1f %% "
- "%10u max %10u abs_max",
+ "%28s %10.3f (%5.1f%%) "
+ "%10u (%5.1f%%) %10u %10u",
name, ms, percent, stats.total_allocated_bytes_,
size_percent, stats.max_allocated_bytes_,
stats.absolute_max_allocated_bytes_);
os << buffer;
if (stats.function_name_.size() > 0) {
- os << " : " << stats.function_name_.c_str();
+ os << " " << stats.function_name_.c_str();
}
os << std::endl;
}
@@ -90,7 +89,10 @@ static void WriteFullLine(std::ostream& os) {
static void WriteHeader(std::ostream& os) {
WriteFullLine(os);
- os << " Turbofan timing results:\n";
+ os << " Turbonfan phase Time (ms) "
+ << " Space (bytes) Function\n"
+ << " "
+ << " Total Max. Abs. max.\n";
WriteFullLine(os);
}
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698