Changeset 26811 in webkit for trunk/JavaScriptCore/kjs/nodes.h

Timestamp:

Oct 20, 2007, 12:11:53 AM (18 years ago)

Author:

ggaren

Message:

Reviewed by Maciej Stachowiak.

Fixed ​http://bugs.webkit.org/show_bug.cgi?id=15570
Store gathered declaration nodes in the function body node.

This means that you only have to gather the declaration nodes the first
time the function executes. Performance gain of 2.10% on SunSpider,
0.90% on command-line JS iBench.

• kjs/nodes.cpp: Split declaration stack initialization code off into initializeDeclarationStacks(). (FunctionBodyNode::FunctionBodyNode): (FunctionBodyNode::initializeDeclarationStacks): (FunctionBodyNode::processDeclarations):

• kjs/nodes.h: Changed DeclarationStacks structure to hold references, since the actual Vectors are now stored either on the stack or in the function body node.

File:

• trunk/JavaScriptCore/kjs/nodes.h (modified) (2 diffs)

trunk/JavaScriptCore/kjs/nodes.h

@@ -90 +90 @@
       typedef Vector<FuncDeclNode*, 16> FunctionStack;
       
-      NodeStack nodeStack;
-      VarStack varStack; 
-      FunctionStack functionStack;
+      DeclarationStacks(NodeStack& n, VarStack& v, FunctionStack& f)
+        : nodeStack(n)
+        , varStack(v)
+        , functionStack(f)
+      {
+      }
+        
+      NodeStack& nodeStack;
+      VarStack& varStack; 
+      FunctionStack& functionStack;
   };
 
@@ -1057 +1064 @@
     int m_sourceId;
     Vector<Identifier> m_parameters;
+
+    void initializeDeclarationStacks();
+    bool m_initializedDeclarationStacks;
+    DeclarationStacks::VarStack m_varStack;
+    DeclarationStacks::FunctionStack m_functionStack;
   };