Skip to content

Commit 8bd6625

Browse files
author
Thomas Preud'homme
committed
Fix runtest/test_suite.shtest test
Commit 81f67a8 added a new code size metric to the CSV report template but did not update the corresponding CHECK-CSV FileCheck directives, leading the test to fail. This commit fixes the problem. Reviewed By: tnfchris Differential Revision: https://reviews.llvm.org/D112182
1 parent 9290733 commit 8bd6625

File tree

2 files changed

+3
-2
lines changed

2 files changed

+3
-2
lines changed

tests/runtest/Inputs/test-suite-cmake/fake-results.json

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@
77
"metrics": {
88
"compile_time": 1.3,
99
"exec_time": 1.4,
10+
"size..text": 42.0,
1011
"score": 1.5,
1112
"hash": "xyz",
1213
"unknown": "unknown"

tests/runtest/test_suite.shtest

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -40,8 +40,8 @@
4040
# CHECK-XML: </testcase>
4141
# CHECK-XML: </testsuite>
4242

43-
# CHECK-CSV: Program;CC;CC_Time;CC_Hash;Exec;Exec_Time;Score
44-
# CHECK-CSV-NEXT: foo//foo;pass;1.3;xyz;pass;1.4;1.5
43+
# CHECK-CSV: Program;CC;CC_Time;Code_Size;CC_Hash;Exec;Exec_Time;Score
44+
# CHECK-CSV-NEXT: foo//foo;pass;1.3;42.0;xyz;pass;1.4;1.5
4545

4646
# CHECK-CHECKFORMAT: Import succeeded.
4747
# CHECK-CHECKFORMAT: Added Machines: 1

0 commit comments

Comments
 (0)