Always write index.html

tools/run_tests/run_microbenchmark.py

@@ -228,38 +228,39 @@ argp.add_argument('--summary_time',
                   help='Minimum time to run benchmarks for the summary collection')
 args = argp.parse_args()
 
-for bm_name in args.benchmarks:
+try:
-  for collect in args.collect:
+  for bm_name in args.benchmarks:
-    collectors[collect](bm_name, args)
+    for collect in args.collect:
-if args.diff_perf:
+      collectors[collect](bm_name, args)
-  if 'summary' not in args.collect:
+  if args.diff_perf:
-    for bm_name in args.benchmarks:
+    if 'summary' not in args.collect:
-      run_summary(bm_name, 'opt', bm_name)
+      for bm_name in args.benchmarks:
-      run_summary(bm_name, 'counters', bm_name)
+        run_summary(bm_name, 'opt', bm_name)
-  where_am_i = subprocess.check_output(['git', 'rev-parse', '--abbrev-ref', 'HEAD']).strip()
+        run_summary(bm_name, 'counters', bm_name)
-  subprocess.check_call(['git', 'checkout', args.diff_perf])
+    where_am_i = subprocess.check_output(['git', 'rev-parse', '--abbrev-ref', 'HEAD']).strip()
-  comparables = []
+    subprocess.check_call(['git', 'checkout', args.diff_perf])
-  subprocess.check_call(['make', 'clean'])
+    comparables = []
-  try:
+    subprocess.check_call(['make', 'clean'])
-    for bm_name in args.benchmarks:
+    try:
-      try:
+      for bm_name in args.benchmarks:
-        run_summary(bm_name, 'opt', '%s.old' % bm_name)
+        try:
-        run_summary(bm_name, 'counters', '%s.old' % bm_name)
+          run_summary(bm_name, 'opt', '%s.old' % bm_name)
-        comparables.append(bm_name)
+          run_summary(bm_name, 'counters', '%s.old' % bm_name)
-      except subprocess.CalledProcessError, e:
+          comparables.append(bm_name)
-        pass
+        except subprocess.CalledProcessError, e:
-  finally:
+          pass
-    subprocess.check_call(['git', 'checkout', where_am_i])
+    finally:
-  for bm_name in comparables:
+      subprocess.check_call(['git', 'checkout', where_am_i])
-    diff = subprocess.check_output(['tools/profiling/microbenchmarks/bm_diff.py',
+    for bm_name in comparables:
-                                    '%s.opt.json' % bm_name,
+      diff = subprocess.check_output(['tools/profiling/microbenchmarks/bm_diff.py',
-                                    '%s.counters.json' % bm_name,
+                                      '%s.opt.json' % bm_name,
-                                    '%s.old.opt.json' % bm_name,
+                                      '%s.counters.json' % bm_name,
-                                    '%s.old.counters.json' % bm_name]).strip()
+                                      '%s.old.opt.json' % bm_name,
-    if diff:
+                                      '%s.old.counters.json' % bm_name]).strip()
-      heading('Performance diff: %s' % bm_name)
+      if diff:
-      text(diff)
+        heading('Performance diff: %s' % bm_name)
-
+        text(diff)
-index_html += "</body>\n</html>\n"
+finally:
-with open('reports/index.html', 'w') as f:
+  index_html += "</body>\n</html>\n"
-  f.write(index_html)
+  with open('reports/index.html', 'w') as f:
+    f.write(index_html)