about summary refs log tree commit diff
diff options
context:
space:
mode:
authorSiddhesh Poyarekar <siddhesh@sourceware.org>2017-09-16 15:23:12 +0530
committerSiddhesh Poyarekar <siddhesh@sourceware.org>2017-09-16 15:23:12 +0530
commit5a6547b7b9e8174a2aa228c4d3c5de7245f720b6 (patch)
tree2dd047fd82e4c55d89ffd240d8c8a4eab7b389bf
parent06b1de237801402c7da327f0a36f4e6aa2f06cc2 (diff)
downloadglibc-5a6547b7b9e8174a2aa228c4d3c5de7245f720b6.tar.gz
glibc-5a6547b7b9e8174a2aa228c4d3c5de7245f720b6.tar.xz
glibc-5a6547b7b9e8174a2aa228c4d3c5de7245f720b6.zip
benchtests: Make compare_strings.py output a bit prettier
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.
-rw-r--r--ChangeLog3
-rwxr-xr-xbenchtests/scripts/compare_strings.py20
2 files changed, 14 insertions, 9 deletions
diff --git a/ChangeLog b/ChangeLog
index fd9cc0ce99..20746e64a9 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 3ca9429d04..1f0be3b981 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('')