[tools] Add Group-Total-V8 category to callstats.py

camillobruni · Commit bot · commit a9ed516ceb0c · 2016-09-23T13:36:55.000Z
Group-Total-V8 contains all but Group-Callback times which should lead to more stable data. R=hablich@chromium.org NOTRY=true BUG= Review-Url: https://codereview.chromium.org/2366523002 Cr-Commit-Position: refs/heads/master@{#39668}
diff --git a/tools/callstats.py b/tools/callstats.py
@@ -317,10 +317,10 @@ def read_stats(path, domain, args):
         ('Group-IC', re.compile(".*IC.*")),
         ('Group-Optimize',
          re.compile("StackGuard|.*Optimize.*|.*Deoptimize.*|Recompile.*")),
-        ('Group-Compile', re.compile(".*Compile.*")),
-        ('Group-Parse', re.compile(".*Parse.*")),
-        ('Group-Callback', re.compile(".*Callback.*")),
-        ('Group-API', re.compile(".*API.*")),
+        ('Group-Compile', re.compile("Compile.*")),
+        ('Group-Parse', re.compile("Parse.*")),
+        ('Group-Callback', re.compile(".*Callback$")),
+        ('Group-API', re.compile("API.*")),
         ('Group-GC', re.compile("GC|AllocateInTargetSpace")),
         ('Group-JavaScript', re.compile("JS_Execution")),
         ('Group-Runtime', re.compile(".*"))]
@@ -355,8 +355,15 @@ def read_stats(path, domain, args):
           entries[group_name]['time'] += time
           entries[group_name]['count'] += count
           break
+    # Calculate the V8-Total (all groups except Callback)
+    total_v8 = { 'time': 0, 'count': 0 }
+    for group_name, regexp in groups:
+      if group_name == 'Group-Callback': continue
+      total_v8['time'] += entries[group_name]['time']
+      total_v8['count'] += entries[group_name]['count']
+    entries['Group-Total-V8'] = total_v8
     # Append the sums as single entries to domain.
-    for key in entries :
+    for key in entries:
       if key not in domain: domain[key] = { 'time_list': [], 'count_list': [] }
       domain[key]['time_list'].append(entries[key]['time'])
       domain[key]['count_list'].append(entries[key]['count'])
