Changeset 34851 in webkit for trunk/JavaScriptCore/VM/Machine.cpp

Timestamp:

Jun 28, 2008, 8:50:49 AM (17 years ago)

Author:

Darin Adler

Message:

2008-06-28 Darin Adler <Darin Adler>

Reviewed by Oliver.

• ​https://bugs.webkit.org/show_bug.cgi?id=19787 create most arrays from values in registers rather than with multiple put operations

SunSpider says 0.8% faster.

• VM/CodeBlock.cpp: (KJS::CodeBlock::dump): Added argv and argc parameters to new_array.
• VM/Machine.cpp: (KJS::Machine::privateExecute): Ditto.

• VM/CodeGenerator.cpp: (KJS::CodeGenerator::emitNewArray): Added.
• VM/CodeGenerator.h: Added ElementNode* argument to emitNewArray.

• kjs/nodes.cpp: (KJS::ArrayNode::emitCode): Pass the ElementNode to emitNewArray so it can be initialized with as many elements as possible. If the array doesn't have any holes in it, that's all that's needed. If there are holes, then emit some separate put operations for the other values in the array and for the length as needed.

• kjs/nodes.h: Added some accessors to ElementNode so the code generator can iterate through elements and generate code to evaluate them. Now ArrayNode does not need to be a friend. Also took out some unused PlacementNewAdoptType constructors.

File:

• trunk/JavaScriptCore/VM/Machine.cpp (modified) (2 diffs)

trunk/JavaScriptCore/VM/Machine.cpp

@@ -36 +36 @@
 #include "ExecState.h"
 #include "JSActivation.h"
+#include "JSArray.h"
+#include "JSFunction.h"
 #include "JSLock.h"
 #include "JSPropertyNameIterator.h"
+#include "JSString.h"
 #include "Parser.h"
 #include "Profiler.h"
+#include "RegExpObject.h"
 #include "Register.h"
-#include "JSArray.h"
 #include "debugger.h"
-#include "JSFunction.h"
-#include "JSString.h"
 #include "object_object.h"
 #include "operations.h"
-#include "RegExpObject.h"
-
 #include <stdio.h>
 
@@ -1037 +1036 @@
     }
     BEGIN_OPCODE(op_new_array) {
-        /* new_array dst(r)
-
-           Constructs a new empty Array instance using the original
+        /* new_array dst(r) firstArg(r) argCount(n)
+
+           Constructs a new Array instance using the original
            constructor, and puts the result in register dst.
-        */
-        int dst = (++vPC)->u.operand;
-        r[dst].u.jsValue = constructEmptyArray(exec);
+           The array will contain argCount elements with values
+           taken from registers starting at register firstArg.
+        */
+        int dst = (++vPC)->u.operand;
+        int firstArg = (++vPC)->u.operand;
+        int argCount = (++vPC)->u.operand;
+        ArgList args(reinterpret_cast<JSValue***>(&registerBase), r - registerBase + firstArg, argCount);
+        r[dst].u.jsValue = constructArray(exec, args);
 
         ++vPC;