diff mbox series

[COMMITTED,2/3] benchtests: Make compare_strings.py output a bit prettier

Message ID 1505556666-3043-2-git-send-email-siddhesh@sourceware.org
State New
Headers show
Series [COMMITTED,1/3] benchtests: Use argparse to parse arguments | expand

Commit Message

Siddhesh Poyarekar Sept. 16, 2017, 10:11 a.m. UTC 
Make the column widths for the outputs fixed so that they look a
little less messy.  They will still look bad with lots of IFUNCs (like
on x86) but it's still a step forward.

	* benchtests/scripts/compare_strings.py (process_results):
	Better spacing for output.
---
 ChangeLog                             |  3 +++
 benchtests/scripts/compare_strings.py | 20 +++++++++++---------
 2 files changed, 14 insertions(+), 9 deletions(-)
diff mbox series

Patch

diff --git a/ChangeLog b/ChangeLog
index fd9cc0c..20746e6 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,8 @@ 
 2017-09-16  Siddhesh Poyarekar  <siddhesh@sourceware.org>
 
+	* benchtests/scripts/compare_strings.py (process_results):
+	Better spacing for output.
+
 	* benchtests/scripts/compare_strings.py: Use argparse.
 	* benchtests/README: Document existence of compare_strings.py.
 
diff --git a/benchtests/scripts/compare_strings.py b/benchtests/scripts/compare_strings.py
index 3ca9429..1f0be3b 100755
--- a/benchtests/scripts/compare_strings.py
+++ b/benchtests/scripts/compare_strings.py
@@ -88,26 +88,28 @@  def process_results(results, attrs, base_func):
 
     for f in results['functions'].keys():
         print('Function: %s' % f)
+        v = results['functions'][f]['bench-variant']
+        print('Variant: %s' % v)
+
         base_index = 0
         if base_func:
             base_index = results['functions'][f]['ifuncs'].index(base_func)
 
-        print('\t'.join(results['functions'][f]['ifuncs']))
-        v = results['functions'][f]['bench-variant']
-        print('Variant: %s' % v)
-        print("=" * 80)
+        print("%36s%s" % (' ', '\t'.join(results['functions'][f]['ifuncs'])))
+        print("=" * 120)
         graph_res = {}
         for res in results['functions'][f]['results']:
             attr_list = ['%s=%s' % (a, res[a]) for a in attrs]
             i = 0
-            key = ','.join(attr_list)
-            sys.stdout.write('%s: \t' % key)
+            key = ', '.join(attr_list)
+            sys.stdout.write('%36s: ' % key)
             graph_res[key] = res['timings']
             for t in res['timings']:
-                sys.stdout.write ('%.2f' % t)
+                sys.stdout.write ('%12.2f' % t)
                 if i != base_index:
-                    diff = (res['timings'][base_index] - t) * 100 / res['timings'][base_index]
-                    sys.stdout.write (' (%.2f%%)' % diff)
+                    base = res['timings'][base_index]
+                    diff = (base - t) * 100 / base
+                    sys.stdout.write (' (%6.2f%%)' % diff)
                 sys.stdout.write('\t')
                 i = i + 1
             print('')