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

Unified Diff: bench/nanobench.cpp

Issue 990723002: DM: display current memory usage (instead of peak) when available. (Closed) Base URL: https://skia.googlesource.com/skia@master
Patch Set: better best Created 5 years, 9 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 | dm/DM.cpp » ('j') | tools/ProcStats.cpp » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: bench/nanobench.cpp
diff --git a/bench/nanobench.cpp b/bench/nanobench.cpp
index d66b308b71e0b7364c6e2d04aef57af908ab280f..920979cbf9e66c865f2aa21922344ebaccc4e262 100644
--- a/bench/nanobench.cpp
+++ b/bench/nanobench.cpp
@@ -476,7 +476,7 @@ public:
if (FLAGS_mpd) {
fUseMPDs.push_back() = true;
}
-
+
// Prepare the images for decoding
for (int i = 0; i < FLAGS_images.count(); i++) {
const char* flag = FLAGS_images[i];
@@ -492,7 +492,7 @@ public:
fImages.push_back() = flag;
}
}
-
+
// Choose the candidate color types for image decoding
const SkColorType colorTypes[] =
{ kN32_SkColorType, kRGB_565_SkColorType, kAlpha_8_SkColorType };
@@ -842,7 +842,7 @@ int nanobench_main() {
config = ""; // Only print the config if we run the same bench on more than one.
}
SkDebugf("%4dM\t%s\t%s\n"
- , sk_tools::getMaxResidentSetSizeMB()
+ , sk_tools::getBestResidentSetSizeMB()
, bench->getUniqueName()
, config);
} else if (FLAGS_verbose) {
@@ -858,7 +858,7 @@ int nanobench_main() {
} else {
const double stddev_percent = 100 * sqrt(stats.var) / stats.mean;
SkDebugf("%4dM\t%d\t%s\t%s\t%s\t%s\t%.0f%%\t%s\t%s\t%s\n"
- , sk_tools::getMaxResidentSetSizeMB()
+ , sk_tools::getBestResidentSetSizeMB()
, loops
, HUMANIZE(stats.min)
, HUMANIZE(stats.median)
« no previous file with comments | « no previous file | dm/DM.cpp » ('j') | tools/ProcStats.cpp » ('J')

Powered by Google App Engine
This is Rietveld 408576698