[llvm-commits] [LNT] r162241 - /lnt/trunk/lnt/tests/nt.py

[David Blaikie]

Author: dblaikie
Date: Mon Aug 20 17:40:10 2012
New Revision: 162241

URL: http://llvm.org/viewvc/llvm-project?rev=162241&view=rev
Log:
Update parsing of Nightly Test runs to choose the appropriate timing metric.

This change is paired with a change to the test-suite. Apologies for any hiccups
(failing LNT runs) from bots that pickup one change without the other.

Reviewed by Daniel Dunbar.

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

Modified: lnt/trunk/lnt/tests/nt.py
URL: http://llvm.org/viewvc/llvm-project/lnt/trunk/lnt/tests/nt.py?rev=162241&r1=162240&r2=162241&view=diff
==============================================================================
--- lnt/trunk/lnt/tests/nt.py (original)
+++ lnt/trunk/lnt/tests/nt.py Mon Aug 20 17:40:10 2012
@@ -285,8 +285,12 @@
     sample_keys = []
     if opts.test_style == "simple":
         test_namespace = 'nts'
-        sample_keys.append(('compile', 'CC_Time', None, 'CC'))
-        sample_keys.append(('exec', 'Exec_Time', None, 'Exec'))
+        time_stat = ''
+        # for now, user time is the unqualified Time stat
+        if opts.test_time_stat == "real":
+            time_stat = 'Real_'
+        sample_keys.append(('compile', 'CC_' + time_stat + 'Time', None, 'CC'))
+        sample_keys.append(('exec', 'Exec_' + time_stat + 'Time', None, 'Exec'))
     else:
         test_namespace = 'nightlytest'
         sample_keys.append(('gcc.compile', 'GCCAS', 'time'))
@@ -1017,6 +1021,10 @@
                          help="Set the test style to run [%default]",
                          choices=('nightly', 'simple'), default='simple')
 
+        group.add_option("", "--test-time-stat", dest="test_time_stat",
+                         help="Set the test timing statistic to gather [%default]",
+                         choices=('user', 'real'), default='user')
+
         group.add_option("", "--disable-cxx", dest="test_cxx",
                          help="Disable C++ tests",
                          action="store_false", default=True)