Ignore:
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:
1 edited

Legend:

Unmodified
Added
Removed

  • trunk/JavaScriptCore/JavaScriptCore.exp

    r43392 r43395  
    115115__ZN3JSC12JSGlobalData6createEb
    116116__ZN3JSC12JSGlobalDataD1Ev
    117 __ZN3JSC12SamplingTool13notifyOfScopeEPNS_9ScopeNodeE
    118117__ZN3JSC12SamplingTool4dumpEPNS_9ExecStateE
    119118__ZN3JSC12SamplingTool5setupEv
Note: See TracChangeset for help on using the changeset viewer.