Commit 81db29dd authored by mtrofin's avatar mtrofin Committed by Commit bot

[tools] Handle benchmark runs reporting more than one result.

The Wasm suite reports both compile time and code size, however,
perf-to-html.py elides one of the results (compile time, in this case).

This affects the effectiveness of results reported by perf try bot runs.

This CL addresses that.

BUG=

Review-Url: https://codereview.chromium.org/1941573002
Cr-Commit-Position: refs/heads/master@{#35935}
parent cbf2a542
...@@ -115,8 +115,8 @@ class Benchmark: ...@@ -115,8 +115,8 @@ class Benchmark:
self.name_ = name self.name_ = name
self.tests_ = {} self.tests_ = {}
for test in data: for test in data:
# strip off "<name>/" prefix # strip off "<name>/" prefix, allowing for subsequent "/"s
test_name = test.split("/")[1] test_name = test.split("/", 1)[1]
self.appendResult(test_name, data[test]) self.appendResult(test_name, data[test])
# tests is a dictionary of Results # tests is a dictionary of Results
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment