Changeset 29067 in webkit for trunk/JavaScriptCore/kjs/nodes.cpp

Timestamp:

Jan 1, 2008, 11:35:37 AM (17 years ago)

Author:

Darin Adler

Message:

Reviewed by Geoff.

• ​http://bugs.webkit.org/show_bug.cgi?id=16685 eliminate List::empty() to cut down on PIC branches

Also included one other speed-up -- remove the call to reserveCapacity from
FunctionBodyNode::processDeclarations in all but the most unusual cases.

Together these make SunSpider 1.016x as fast.

• JavaScriptCore.exp: Updated.
• kjs/ExecState.cpp: (KJS::globalEmptyList): Added. Called only when creating global ExecState instances. (KJS::ExecState::ExecState): Broke constructor up into three separate functions, for the three separate node types. Also went through each of the three and streamlined as much as possible, removing dead code. This prevents us from having to access the global in the function body version of the constructor.

• kjs/ExecState.h: Added emptyList(). Replaced the constructor with a set of three that are specific to the different node types that can create new execution state objects.

• kjs/array_object.cpp: (KJS::ArrayProtoFuncToLocaleString::callAsFunction): Use exec->emptyList() instead of List::empty(). (KJS::ArrayProtoFuncConcat::callAsFunction): Ditto. (KJS::ArrayProtoFuncSlice::callAsFunction): Ditto. (KJS::ArrayProtoFuncSplice::callAsFunction): Ditto. (KJS::ArrayProtoFuncFilter::callAsFunction): Ditto.
• kjs/function.cpp: (KJS::FunctionImp::callAsFunction): Updated to call new ExecState constructor. (KJS::GlobalFuncImp::callAsFunction): Ditto (for eval).
• kjs/function_object.cpp: (FunctionObjectImp::construct): Use exec->emptyList() instead of List::empty().

• kjs/list.cpp: Removed List::empty.
• kjs/list.h: Ditto.

• kjs/nodes.cpp: (KJS::ElementNode::evaluate): Use exec->emptyList() instead of List::empty(). (KJS::ArrayNode::evaluate): Ditto. (KJS::ObjectLiteralNode::evaluate): Ditto. (KJS::PropertyListNode::evaluate): Ditto. (KJS::FunctionBodyNode::processDeclarations): Another speed-up. Check the capacity before calling reserveCapacity, because it doesn't get inlined the local storage vector is almost always big enough -- saving the function call overhead is a big deal. (KJS::FuncDeclNode::makeFunction): Use exec->emptyList() instead of List::empty(). (KJS::FuncExprNode::evaluate): Ditto.
• kjs/object.cpp: (KJS::tryGetAndCallProperty): Ditto.
• kjs/property_slot.cpp: (KJS::PropertySlot::functionGetter): Ditto.
• kjs/string_object.cpp: (KJS::StringProtoFuncSplit::callAsFunction): Ditto.

File:

• trunk/JavaScriptCore/kjs/nodes.cpp (modified) (7 diffs)

trunk/JavaScriptCore/kjs/nodes.cpp

@@ -636 +636 @@
 JSValue *ElementNode::evaluate(ExecState *exec)
 {
-  JSObject *array = exec->lexicalGlobalObject()->arrayConstructor()->construct(exec, List::empty());
+  JSObject* array = exec->lexicalGlobalObject()->arrayConstructor()->construct(exec, exec->emptyList());
   int length = 0;
   for (ElementNode *n = this; n; n = n->next.get()) {
@@ -667 +667 @@
     length = opt ? array->get(exec, exec->propertyNames().length)->toInt32(exec) : 0;
   } else {
-    JSValue *newArr = exec->lexicalGlobalObject()->arrayConstructor()->construct(exec,List::empty());
+    JSValue* newArr = exec->lexicalGlobalObject()->arrayConstructor()->construct(exec, exec->emptyList());
     array = static_cast<JSObject*>(newArr);
     length = 0;
@@ -692 +692 @@
     return list->evaluate(exec);
 
-  return exec->lexicalGlobalObject()->objectConstructor()->construct(exec,List::empty());
+  return exec->lexicalGlobalObject()->objectConstructor()->construct(exec, exec->emptyList());
 }
 
@@ -707 +707 @@
 JSValue *PropertyListNode::evaluate(ExecState *exec)
 {
-  JSObject *obj = exec->lexicalGlobalObject()->objectConstructor()->construct(exec, List::empty());
+  JSObject* obj = exec->lexicalGlobalObject()->objectConstructor()->construct(exec, exec->emptyList());
   
   for (PropertyListNode *p = this; p; p = p->next.get()) {
@@ -4362 +4362 @@
     // We can't just resize localStorage here because that would temporarily
     // leave uninitialized entries, which would crash GC during the mark phase.
-    localStorage.reserveCapacity(m_varStack.size() + m_parameters.size() + m_functionStack.size());
+    size_t totalSize = m_varStack.size() + m_parameters.size() + m_functionStack.size();
+    if (totalSize > localStorage.capacity()) // Doing this check inline avoids function call overhead.
+        localStorage.reserveCapacity(totalSize);
     
     int minAttributes = Internal | DontDelete;
@@ -4544 +4546 @@
   FunctionImp *func = new FunctionImp(exec, ident, body.get(), exec->scopeChain());
 
-  JSObject *proto = exec->lexicalGlobalObject()->objectConstructor()->construct(exec, List::empty());
+  JSObject* proto = exec->lexicalGlobalObject()->objectConstructor()->construct(exec, exec->emptyList());
   proto->put(exec, exec->propertyNames().constructor, func, ReadOnly | DontDelete | DontEnum);
   func->put(exec, exec->propertyNames().prototype, proto, Internal|DontDelete);
@@ -4580 +4582 @@
 
   FunctionImp* func = new FunctionImp(exec, ident, body.get(), exec->scopeChain());
-  JSObject* proto = exec->lexicalGlobalObject()->objectConstructor()->construct(exec, List::empty());
+  JSObject* proto = exec->lexicalGlobalObject()->objectConstructor()->construct(exec, exec->emptyList());
   proto->put(exec, exec->propertyNames().constructor, func, ReadOnly | DontDelete | DontEnum);
   func->put(exec, exec->propertyNames().prototype, proto, Internal | DontDelete);