Changeset 28929 in webkit for trunk/JavaScriptCore/kjs/nodes2string.cpp

Timestamp:

Dec 20, 2007, 11:21:20 PM (17 years ago)

Author:

[email protected]

Message:

2007-12-17 Eric Seidel <​[email protected]>

Reviewed by mjs.

Split IfNode into IfNode and IfElseNode for speedup.
​http://bugs.webkit.org/show_bug.cgi?id=16470

SunSpider claims this is 1.003x as fast as before.
(This required running with --runs 15 to get consistent enough results to tell!)

• kjs/grammar.y:
• kjs/nodes.cpp: (KJS::IfNode::optimizeVariableAccess): (KJS::IfNode::execute): (KJS::IfNode::getDeclarations): (KJS::IfElseNode::optimizeVariableAccess): (KJS::IfElseNode::execute): (KJS::IfElseNode::getDeclarations):
• kjs/nodes.h: (KJS::IfNode::): (KJS::IfElseNode::):
• kjs/nodes2string.cpp: (KJS::IfNode::streamTo): (KJS::IfElseNode::streamTo):

File:

• trunk/JavaScriptCore/kjs/nodes2string.cpp (modified) (1 diff)

trunk/JavaScriptCore/kjs/nodes2string.cpp

@@ -753 +753 @@
 void IfNode::streamTo(SourceStream& s) const
 {
-    s << Endl << "if (" << expr << ')' << Indent << statement1 << Unindent;
-    if (statement2)
-        s << Endl << "else" << Indent << statement2 << Unindent;
+    s << Endl << "if (" << m_condition << ')' << Indent << m_ifBlock << Unindent;
+}
+
+void IfElseNode::streamTo(SourceStream& s) const
+{
+    IfNode::streamTo(s);
+    s << Endl << "else" << Indent << m_elseBlock << Unindent;
 }