[LNT] r268592 - Collect cmake test-suite link time

Chris Matthews via llvm-commits llvm-commits at lists.llvm.org
Wed May 4 19:00:38 PDT 2016


Author: cmatthews
Date: Wed May  4 21:00:37 2016
New Revision: 268592

URL: http://llvm.org/viewvc/llvm-project?rev=268592&view=rev
Log:
Collect cmake test-suite link time

The test suite results include link time, but LNT was dropping that. I
think for now it is better to not introduce a new sampile kind for this,
so I hotwired it into the benchmark name, so "nts.foo/bar.compile" now
will also have nts.foo/bar-link.compile

Modified:
    lnt/trunk/lnt/tests/test_suite.py

Modified: lnt/trunk/lnt/tests/test_suite.py
URL: http://llvm.org/viewvc/llvm-project/lnt/trunk/lnt/tests/test_suite.py?rev=268592&r1=268591&r2=268592&view=diff
==============================================================================
--- lnt/trunk/lnt/tests/test_suite.py (original)
+++ lnt/trunk/lnt/tests/test_suite.py Wed May  4 21:00:37 2016
@@ -563,13 +563,15 @@ class TestSuiteTest(BuiltinTest):
             'compile_time': 'compile',
             'exec_time': 'exec',
             'score': 'score',
-            'hash': 'hash'
+            'hash': 'hash',
+            'link_time': 'compile'
         }
         LIT_METRIC_CONV_FN = {
             'compile_time': float,
             'exec_time': float,
             'score': float,
-            'hash': str
+            'hash': str,
+            'link_time': float
         }
         
         # We don't use the test info, currently.
@@ -605,11 +607,17 @@ class TestSuiteTest(BuiltinTest):
                     if k == 'profile':
                         profiles_to_import.append((name, v))
                         continue
-                    
+
                     if k not in LIT_METRIC_TO_LNT or LIT_METRIC_TO_LNT[k] in ignore:
                         continue
+                    server_name = name + '.' + LIT_METRIC_TO_LNT[k]
+
+                    if k == 'link_time':
+                        # Move link time into a second benchmark's compile-time.
+                        server_name =  name + '-link.' + LIT_METRIC_TO_LNT[k]
+    
                     test_samples.append(
-                        lnt.testing.TestSamples(name + '.' + LIT_METRIC_TO_LNT[k],
+                        lnt.testing.TestSamples(server_name,
                                                 [v],
                                                 test_info,
                                                 LIT_METRIC_CONV_FN[k]))




More information about the llvm-commits mailing list