Changeset 38420 in webkit for trunk/JavaScriptCore/parser/Nodes.h

Timestamp:

Nov 15, 2008, 12:22:22 AM (17 years ago)

Author:

[email protected]

Message:

2008-11-15 Maciej Stachowiak <​[email protected]>

Reviewed by Sam Weinig.

• Remove SymbolTable from FunctionBodyNode and move it to CodeBlock

It's not needed for functions that have never been executed, so no
need to waste the memory. Saves ~4M on membuster after 30 pages.

• VM/CodeBlock.h:
• VM/Machine.cpp: (JSC::Machine::retrieveArguments):
• parser/Nodes.cpp: (JSC::EvalNode::generateCode): (JSC::FunctionBodyNode::generateCode):
• parser/Nodes.h:
• runtime/JSActivation.h: (JSC::JSActivation::JSActivationData::JSActivationData):

File:

• trunk/JavaScriptCore/parser/Nodes.h (modified) (2 diffs)

trunk/JavaScriptCore/parser/Nodes.h

@@ -2153 +2153 @@
         virtual RegisterID* emitCode(CodeGenerator&, RegisterID* = 0) JSC_FAST_CALL;
         
-        SymbolTable& symbolTable() { return m_symbolTable; } // FIXME: Remove this
-        
         CodeBlock& byteCode(ScopeChainNode* scopeChain) JSC_FAST_CALL
         {
@@ -2202 +2200 @@
         Identifier* m_parameters;
         size_t m_parameterCount;
-        SymbolTable m_symbolTable;
         OwnPtr<CodeBlock> m_code;
         unsigned m_refCount;