This is the mail archive of the
libc-alpha@sourceware.org
mailing list for the glibc project.
[COMMITTED 2/3] benchtests: Make compare_strings.py output a bit prettier
- From: Siddhesh Poyarekar <siddhesh at sourceware dot org>
- To: libc-alpha at sourceware dot org
- Date: Sat, 16 Sep 2017 15:41:05 +0530
- Subject: [COMMITTED 2/3] benchtests: Make compare_strings.py output a bit prettier
- Authentication-results: sourceware.org; auth=none
- References: <1505556666-3043-1-git-send-email-siddhesh@sourceware.org>
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 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('')
--
2.7.4