From 356d1a5d65b0c21b6dce57aca842e73458f6873a Mon Sep 17 00:00:00 2001 From: David Horstmann Date: Mon, 2 Sep 2019 14:29:54 +0100 Subject: Fix run.py result calculation when using the --dont-auto-calibrate option. tools/benchmarks/run.py currently only records results from output lines that end with 'per iteration'. These do not appear when the benchmarks are run with the --dont-auto-calibrate option as no bench runner is used. Instead, allow any line that starts with 'benchmarks/' to be parsed. This prevents a crash that happens when run.py tries to calculate the geomean while running with --dont-auto-calibrate. Test: tools/benchmarks/run.py --iterations 1 --dont-auto-calibrate Change-Id: I179a7c1b4a98288757bebcc721f1d4f6bd7b05f1 --- tools/benchmarks/run.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'tools') diff --git a/tools/benchmarks/run.py b/tools/benchmarks/run.py index 2a90f3a..c3ea63f 100755 --- a/tools/benchmarks/run.py +++ b/tools/benchmarks/run.py @@ -164,7 +164,7 @@ def RunBench(apk, classname, try: for line in outerr.rstrip().splitlines(): - if not line.endswith('per iteration'): + if not line.startswith('benchmarks/'): continue name = line.split(":")[0].rstrip() score = float(line.split(":")[1].strip().split(" ")[0].strip()) -- cgit v1.2.3