Changeset 43395 in webkit for trunk/JavaScriptCore/bytecode/SamplingTool.h

Timestamp:

May 8, 2009, 3:18:34 AM (16 years ago)

Author:

[email protected]

Message:

2009-05-08 Gavin Barraclough <​[email protected]>

Reviewed by Oliver "I see lots of ifdefs" Hunt.

Fix (kinda) for sampling tool breakage. The codeblock sampling tool has become
b0rked due to recent changes in native function calling. The initialization of
a ScopeNode appears to now occur before the sampling tool (or possibly the
interpreter has been brought into existence, wihich leads to crashyness).

This patch doesn't fix the problem. The crash occurs when tracking a Scope, but
we shouldn't need to track scopes when we're just sampling opcodes, not
codeblocks. Not retaining Scopes when just opcode sampling will reduce sampling
overhead reducing any instrumentation skew, which is a good thing. As a side
benefit this patch also gets the opcode sampling going again, albeit in a bit of
a lame way. Will come back later with a proper fix from codeblock sampling.

• JavaScriptCore.exp:
• bytecode/SamplingTool.cpp: (JSC::compareLineCountInfoSampling): (JSC::SamplingTool::dump):
• bytecode/SamplingTool.h: (JSC::SamplingTool::SamplingTool):
• parser/Nodes.cpp: (JSC::ScopeNode::ScopeNode):

File:

• trunk/JavaScriptCore/bytecode/SamplingTool.h (modified) (3 diffs)

trunk/JavaScriptCore/bytecode/SamplingTool.h

@@ -192 +192 @@
             , m_sampleCount(0)
             , m_opcodeSampleCount(0)
+#if ENABLE(CODEBLOCK_SAMPLING)
             , m_scopeSampleMap(new ScopeSampleRecordMap())
+#endif
         {
             memset(m_opcodeSamples, 0, sizeof(m_opcodeSamples));
@@ -198 +200 @@
         }
 
+#if ENABLE(CODEBLOCK_SAMPLING)
         ~SamplingTool()
         {
             deleteAllValues(*m_scopeSampleMap);
         }
+
+        void notifyOfScope(ScopeNode* scope);
+#endif
 
         void setup();
         void dump(ExecState*);
-
-        void notifyOfScope(ScopeNode* scope);
 
         void sample(CodeBlock* codeBlock, Instruction* vPC)
@@ -261 +265 @@
         unsigned m_opcodeSamplesInCTIFunctions[numOpcodeIDs];
         
+#if ENABLE(CODEBLOCK_SAMPLING)
         Mutex m_scopeSampleMapMutex;
         OwnPtr<ScopeSampleRecordMap> m_scopeSampleMap;
+#endif
     };