diff options
| author | cmtice <cmtice@google.com> | 2013-10-23 11:44:51 -0700 |
|---|---|---|
| committer | chrome-internal-fetch <chrome-internal-fetch@google.com> | 2013-10-23 21:27:57 +0000 |
| commit | a1d03083c46175d9a34ea3665a3b1e305607442b (patch) | |
| tree | de2bef5d1160530e739547ede7ef345f07885b63 /crosperf/suite_runner.py | |
| parent | 9c956f58876917af6cf5c6961241ddeaf1baf826 (diff) | |
| download | toolchain-utils-a1d03083c46175d9a34ea3665a3b1e305607442b.tar.gz | |
Accept hyphens in test names; fix profiler args string.
Updated regular expression parser to accept hyphens in the test names.
Also fixed the quotes on profiler args, for proper parsing by
test_that.
BUG=None
TEST=Ran crosperf successfully with benchmark name that had hyphen in it;
successfully ran crosperf with profiler args.
Change-Id: I4a154e390a5fabfc47e0783212fed928eb5e9873
Reviewed-on: https://chrome-internal-review.googlesource.com/146967
Reviewed-by: Yunlian Jiang <yunlian@google.com>
Commit-Queue: Caroline Tice <cmtice@google.com>
Tested-by: Caroline Tice <cmtice@google.com>
Diffstat (limited to 'crosperf/suite_runner.py')
| -rw-r--r-- | crosperf/suite_runner.py | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/crosperf/suite_runner.py b/crosperf/suite_runner.py index 47e29c49..f69f3728 100644 --- a/crosperf/suite_runner.py +++ b/crosperf/suite_runner.py @@ -20,7 +20,7 @@ def GetProfilerArgs (benchmark): if not perf_args_list[0] in ["record", "stat"]: raise Exception("perf_args must start with either record or stat") extra_test_args = ["profiler=custom_perf", - ("profiler_args=\"'%s'\"" % + ("profiler_args='%s'" % perf_args)] return " ".join(extra_test_args) else: |