[PATCH] D109576: [LNT] Fixed missing code_size metric

[Pavel Kosov via Phabricator via llvm-commits]

kpdev42 created this revision.
kpdev42 added reviewers: cmatthews, thopre, danilaml.
kpdev42 added a project: LLVM.
Herald added a subscriber: dkolesnichenko.
kpdev42 requested review of this revision.

Fixed missing code_size metric in the final report


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D109576

Files:
  lnt/tests/test_suite.py


Index: lnt/tests/test_suite.py
===================================================================
--- lnt/tests/test_suite.py
+++ lnt/tests/test_suite.py
@@ -73,14 +73,15 @@
 """
 
 CSV_REPORT_TEMPLATE = """\
-Program;CC;CC_Time;CC_Hash;Exec;Exec_Time;Score
+Program;CC;CC_Time;Code_Size;CC_Hash;Exec;Exec_Time;Score
 {%- for suite in suites -%}
     {%- for test in suite.tests %}
 {{ suite.name }}/{{ test.path }}/{{ test.name }};
         {%- if test.code == "NOEXE" -%}
-            fail;*;*;
+            fail;*;*;*;
         {%- else -%}
             pass;{{ test.metrics.compile_time if test.metrics }};\
+{{ test.metrics['size..text'] if test.metrics }};\
 {{ test.metrics.hash if test.metrics }};
         {%- endif -%}
         {%- if test.code == "FAIL" or test.code == "NOEXE" -%}
@@ -713,7 +714,7 @@
             'score': 'score',
             'hash': 'hash',
             'link_time': 'compile',
-            'size.__text': 'code_size',
+            'size..text': 'code_size',
             'mem_bytes': 'mem',
             'link_mem_bytes': 'mem'
         }
@@ -723,7 +724,7 @@
             'score': float,
             'hash': str,
             'link_time': float,
-            'size.__text': float,
+            'size..text': float,
             'mem_bytes': float,
             'link_mem_bytes': float
         }


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D109576.371811.patch
Type: text/x-patch
Size: 1331 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210910/8955b5d4/attachment.bin>