Changeset 43559 in webkit for trunk/JavaScriptCore/parser/Nodes.cpp

Timestamp:

May 12, 2009, 1:58:56 AM (16 years ago)

Author:

[email protected]

Message:

Improve function call forwarding performance

Reviewed by Gavin Barraclough

Make creation of the Arguments object occur lazily, so it
is not necessarily created for every function that references
it. Then add logic to Function.apply to allow it to avoid
allocating the Arguments object at all. Helps a lot with
the function forwarding/binding logic in jQuery, Prototype,
and numerous other JS libraries.

File:

• trunk/JavaScriptCore/parser/Nodes.cpp (modified) (1 diff)

trunk/JavaScriptCore/parser/Nodes.cpp

@@ -460 +460 @@
             generator.emitNode(thisRegister.get(), m_args->m_listNode->m_expr);
             ArgumentListNode* args = m_args->m_listNode->m_next;
-            generator.emitNode(argsRegister.get(), args->m_expr);
+            bool isArgumentsApply = false;
+            if (args->m_expr->isResolveNode()) {
+                ResolveNode* resolveNode = static_cast<ResolveNode*>(args->m_expr);
+                isArgumentsApply = generator.willResolveToArguments(resolveNode->identifier());
+                if (isArgumentsApply)
+                    generator.emitMove(argsRegister.get(), generator.uncheckedRegisterForArguments());
+            }
+            if (!isArgumentsApply)
+                generator.emitNode(argsRegister.get(), args->m_expr);
             while ((args = args->m_next))
                 generator.emitNode(args->m_expr);