Changeset 121382 in webkit for trunk/Source/JavaScriptCore

Timestamp:

Jun 27, 2012, 4:16:10 PM (13 years ago)

Author:

[email protected]

Message:

DFG disassembly should be easier to read
​https://bugs.webkit.org/show_bug.cgi?id=90106

Reviewed by Mark Hahnenberg.

Did a few things:

• Options::showDFGDisassembly now shows OSR exit disassembly as well.

• Phi node dumping doesn't attempt to do line wrapping since it just made the dump harder to read.

• DFG graph disassembly view shows a few additional node types that turn out to be essential for understanding OSR exits.

Put together, these changes reinforce the philosophy that anything needed for computing
OSR exit is just as important as the machine code itself. Of course, we still don't take
that philosophy to its full extreme - for example Phantom nodes are not dumped. We may
revisit that in the future.

• assembler/LinkBuffer.cpp:

(JSC::LinkBuffer::finalizeCodeWithDisassembly):

• assembler/LinkBuffer.h:

(JSC):

• dfg/DFGDisassembler.cpp:

(JSC::DFG::Disassembler::dump):

• dfg/DFGGraph.cpp:

(JSC::DFG::Graph::dumpBlockHeader):

• dfg/DFGNode.h:

(JSC::DFG::Node::willHaveCodeGenOrOSR):

• dfg/DFGOSRExitCompiler.cpp:
• jit/JIT.cpp:

(JSC::JIT::privateCompile):

Location:

trunk/Source/JavaScriptCore

Files:

• ChangeLog (modified) (1 diff)
• assembler/LinkBuffer.cpp (modified) (1 diff)
• assembler/LinkBuffer.h (modified) (2 diffs)
• dfg/DFGDisassembler.cpp (modified) (2 diffs)
• dfg/DFGGraph.cpp (modified) (2 diffs)
• dfg/DFGNode.h (modified) (1 diff)
• dfg/DFGOSRExitCompiler.cpp (modified) (2 diffs)
• jit/JIT.cpp (modified) (1 diff)

trunk/Source/JavaScriptCore/ChangeLog

@@ +1 @@
+2012-06-27  Filip Pizlo  <[email protected]>
+
+        DFG disassembly should be easier to read
+        https://bugs.webkit.org/show_bug.cgi?id=90106
+
+        Reviewed by Mark Hahnenberg.
+        
+        Did a few things:
+        
+        - Options::showDFGDisassembly now shows OSR exit disassembly as well.
+        
+        - Phi node dumping doesn't attempt to do line wrapping since it just made the dump harder
+          to read.
+        
+        - DFG graph disassembly view shows a few additional node types that turn out to be
+          essential for understanding OSR exits.
+        
+        Put together, these changes reinforce the philosophy that anything needed for computing
+        OSR exit is just as important as the machine code itself. Of course, we still don't take
+        that philosophy to its full extreme - for example Phantom nodes are not dumped. We may
+        revisit that in the future.
+
+        * assembler/LinkBuffer.cpp:
+        (JSC::LinkBuffer::finalizeCodeWithDisassembly):
+        * assembler/LinkBuffer.h:
+        (JSC):
+        * dfg/DFGDisassembler.cpp:
+        (JSC::DFG::Disassembler::dump):
+        * dfg/DFGGraph.cpp:
+        (JSC::DFG::Graph::dumpBlockHeader):
+        * dfg/DFGNode.h:
+        (JSC::DFG::Node::willHaveCodeGenOrOSR):
+        * dfg/DFGOSRExitCompiler.cpp:
+        * jit/JIT.cpp:
+        (JSC::JIT::privateCompile):
+
 2012-06-25  Mark Hahnenberg  <[email protected]>
 

trunk/Source/JavaScriptCore/assembler/LinkBuffer.cpp

@@ -42 +42 @@
 LinkBuffer::CodeRef LinkBuffer::finalizeCodeWithDisassembly(const char* format, ...)
 {
-    ASSERT(Options::showDisassembly);
+    ASSERT(Options::showDisassembly || Options::showDFGDisassembly);
     
     CodeRef result = finalizeCodeWithoutDisassembly();

trunk/Source/JavaScriptCore/assembler/LinkBuffer.h

@@ -258 +258 @@
 };
 
+#define FINALIZE_CODE_IF(condition, linkBufferReference, dataLogArgumentsForHeading)  \
+    (UNLIKELY((condition))                                              \
+     ? ((linkBufferReference).finalizeCodeWithDisassembly dataLogArgumentsForHeading) \
+     : (linkBufferReference).finalizeCodeWithoutDisassembly())
+
 // Use this to finalize code, like so:
 //
@@ -275 +280 @@
 
 #define FINALIZE_CODE(linkBufferReference, dataLogArgumentsForHeading)  \
-    (UNLIKELY(Options::showDisassembly)                                 \
-     ? ((linkBufferReference).finalizeCodeWithDisassembly dataLogArgumentsForHeading) \
-     : (linkBufferReference).finalizeCodeWithoutDisassembly())
+    FINALIZE_CODE_IF(Options::showDisassembly, linkBufferReference, dataLogArgumentsForHeading)
 
 } // namespace JSC

trunk/Source/JavaScriptCore/dfg/DFGDisassembler.cpp

@@ -44 +44 @@
     m_graph.m_dominators.computeIfNecessary(m_graph);
     
-    dataLog("Generated JIT code for DFG CodeBlock %p:\n", m_graph.m_codeBlock);
+    dataLog("Generated JIT code for DFG CodeBlock %p, instruction count = %u:\n", m_graph.m_codeBlock, m_graph.m_codeBlock->instructionCount());
     dataLog("    Code at [%p, %p):\n", linkBuffer.debugAddress(), static_cast<char*>(linkBuffer.debugAddress()) + linkBuffer.debugSize());
     
@@ -60 +60 @@
         NodeIndex lastNodeIndexForDisassembly = block->at(0);
         for (size_t i = 0; i < block->size(); ++i) {
-            if (!m_graph[block->at(i)].willHaveCodeGen())
+            if (!m_graph[block->at(i)].willHaveCodeGenOrOSR())
                 continue;
             MacroAssembler::Label currentLabel;

trunk/Source/JavaScriptCore/dfg/DFGGraph.cpp

@@ -328 +328 @@
     }
     dataLog("%s  Phi Nodes:", prefix);
-    unsigned count = 0;
     for (size_t i = 0; i < block->phis.size(); ++i) {
         NodeIndex phiNodeIndex = block->phis[i];
@@ -334 +333 @@
         if (!phiNode.shouldGenerate() && phiNodeDumpMode == DumpLivePhisOnly)
             continue;
-        if (!((++count) % 4))
-            dataLog("\n%s      ", prefix);
         dataLog(" @%u->(", phiNodeIndex);
         if (phiNode.child1()) {

trunk/Source/JavaScriptCore/dfg/DFGNode.h

@@ -732 +732 @@
     }
     
-    bool willHaveCodeGen()
-    {
-        return shouldGenerate() && op() != Phantom && op() != Nop;
+    bool willHaveCodeGenOrOSR()
+    {
+        switch (op()) {
+        case SetLocal:
+        case Int32ToDouble:
+        case ValueToInt32:
+        case UInt32ToNumber:
+        case DoubleAsInt32:
+            return true;
+        case Phantom:
+        case Nop:
+            return false;
+        default:
+            return shouldGenerate();
+        }
     }
 

trunk/Source/JavaScriptCore/dfg/DFGOSRExitCompiler.cpp

@@ -30 +30 @@
 
 #include "CallFrame.h"
+#include "DFGCommon.h"
 #include "LinkBuffer.h"
 #include "RepatchBuffer.h"
@@ -80 +81 @@
         
         LinkBuffer patchBuffer(*globalData, &jit, codeBlock);
-        exit.m_code = FINALIZE_CODE(
+        exit.m_code = FINALIZE_CODE_IF(
+            shouldShowDisassembly(),
             patchBuffer,
             ("DFG OSR exit #%u (bc#%u, @%u, %s) from CodeBlock %p",

trunk/Source/JavaScriptCore/jit/JIT.cpp

@@ -764 +764 @@
     
     CodeRef result = FINALIZE_CODE(
-        patchBuffer, ("Baseline JIT code for CodeBlock %p", m_codeBlock));
+        patchBuffer,
+        ("Baseline JIT code for CodeBlock %p, instruction count = %u",
+         m_codeBlock, m_codeBlock->instructionCount()));
     
     m_globalData->machineCodeBytesPerBytecodeWordForBaselineJIT.add(