From 5a6547b7b9e8174a2aa228c4d3c5de7245f720b6 Mon Sep 17 00:00:00 2001 From: Siddhesh Poyarekar Date: Sat, 16 Sep 2017 15:23:12 +0530 Subject: 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. --- ChangeLog | 3 +++ benchtests/scripts/compare_strings.py | 20 +++++++++++--------- 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 + * 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('') -- cgit 1.4.1