Changeset 195409 in webkit

Timestamp:

Jan 21, 2016, 7:47:31 AM (11 years ago)

Author:

fpizlo@apple.com

Message:

display-profiler-output should be able to display code blocks sorted by machine counts
​https://bugs.webkit.org/show_bug.cgi?id=153298

Reviewed by Oliver Hunt.

• Scripts/display-profiler-output:

Location:

trunk/Tools

Files:

• ChangeLog (modified) (1 diff)
• Scripts/display-profiler-output (modified) (3 diffs)

trunk/Tools/ChangeLog

@@ +1 @@
+2016-01-21  Filip Pizlo  <fpizlo@apple.com>
+
+        display-profiler-output should be able to display code blocks sorted by machine counts
+        https://bugs.webkit.org/show_bug.cgi?id=153298
+
+        Reviewed by Oliver Hunt.
+
+        * Scripts/display-profiler-output:
+
 2016-01-21  Julien Isorce  <j.isorce@samsung.com>
 

trunk/Tools/Scripts/display-profiler-output

@@ -536 +536 @@
 end
 
-def summary(mode)
+def summary(mode, order)
     remaining = screenWidth
     
@@ -619 +619 @@
     $bytecodes.sort {
         | a, b |
-        b.totalMaxTopExecutionCount <=> a.totalMaxTopExecutionCount
+        case order
+        when :bytecode
+            b.totalMaxTopExecutionCount <=> a.totalMaxTopExecutionCount
+        when :machine
+            b.totalMaxBottomExecutionCount <=> a.totalMaxBottomExecutionCount
+        else
+            raise
+        end
     }.each {
         | bytecode |
@@ -739 +746 @@
         exit 0
     when "summary", "s"
-        summary(:summary)
+        summary(:summary, :bytecode)
     when "full", "f"
-        summary(:full)
+        if args[0] and (args[0] == "m" or args[0] == "machine")
+            summary(:full, :machine)
+        else
+            summary(:full, :bytecode)
+        end
     when "source"
         if args.length != 1