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.

diff --git a/ChangeLog b/ChangeLog
index fd9cc0ce99ce7e9c4e9a19f51b89096f227afe65..20746e64a94c922b78d428fe376ccb2646cc69af 100644 (file)
--- 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 3ca9429d048e2111db6eab41f450d4d359542b1d..1f0be3b981ea0e1cdb32f92439400d4c47c4d28a 100755 (executable)
--- 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('')