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

Timestamp:

Oct 25, 2007, 1:11:31 AM (18 years ago)

Author:

ggaren

Message:

Reviewed by Maciej Stachowiak.

Make a fast path for declaration processing inside Function Code.

Lifted declaration processing code up from individual declaration nodes
and into processDeclarations.

Broke out processDeclarations into two cases, depending on the type of
code. This eliminates 2 branches, and facilitates more radical
divergeance in the future.

2.5% SunSpider speedup.

• JavaScriptCore.xcodeproj/project.pbxproj:
• kjs/nodes.cpp: (KJS::FunctionBodyNode::initializeDeclarationStacks): (KJS::FunctionBodyNode::processDeclarationsFunctionCode): (KJS::FunctionBodyNode::processDeclarationsProgramCode): (KJS::FunctionBodyNode::execute): (KJS::FuncDeclNode::makeFunction):
• kjs/nodes.h: (KJS::):

File:

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

trunk/JavaScriptCore/kjs/nodes.h

@@ -1005 +1005 @@
     JSValue* evaluate(ExecState*) KJS_FAST_CALL;
     virtual void streamTo(SourceStream&) const KJS_FAST_CALL;
-    ALWAYS_INLINE void processDeclaration(ExecState*) KJS_FAST_CALL;
     virtual void getDeclarations(DeclarationStacks&) KJS_FAST_CALL;
-  private:
-    JSValue* handleSlowCase(ExecState*, const ScopeChain&, JSValue*) KJS_FAST_CALL KJS_NO_INLINE;
     Type varType;
     Identifier ident;
+  private:
+    JSValue* handleSlowCase(ExecState*, const ScopeChain&, JSValue*) KJS_FAST_CALL KJS_NO_INLINE;
     RefPtr<AssignExprNode> init;
   };
@@ -1242 +1241 @@
     UString paramString() const KJS_FAST_CALL;
     Vector<Identifier>& parameters() KJS_FAST_CALL { return m_parameters; }
-    ALWAYS_INLINE void processDeclarations(ExecState*) KJS_FAST_CALL;
+    ALWAYS_INLINE void processDeclarations(ExecState*);
+    ALWAYS_INLINE void processDeclarationsFunctionCode(ExecState*);
+    ALWAYS_INLINE void processDeclarationsProgramCode(ExecState*);
   private:
     UString m_sourceURL;
@@ -1250 +1251 @@
     bool m_initializedDeclarationStacks;
 
+    // Properties that will go into the ActivationImp's symbol table. (Used for initializing the ActivationImp.)
     DeclarationStacks::VarStack m_varStack;
     DeclarationStacks::FunctionStack m_functionStack;
@@ -1279 +1281 @@
     virtual void streamTo(SourceStream&) const KJS_FAST_CALL;
     virtual void getDeclarations(DeclarationStacks&) KJS_FAST_CALL;
-    ALWAYS_INLINE void processDeclaration(ExecState*) KJS_FAST_CALL;
+    ALWAYS_INLINE FunctionImp* makeFunction(ExecState*) KJS_FAST_CALL;
+    Identifier ident;
   private:
     void addParams() KJS_FAST_CALL;
-    Identifier ident;
     RefPtr<ParameterNode> param;
     RefPtr<FunctionBodyNode> body;