Changeset 34851 in webkit for trunk/JavaScriptCore/VM/CodeBlock.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/CodeBlock.cpp (modified) (2 diffs)

trunk/JavaScriptCore/VM/CodeBlock.cpp

@@ -31 +31 @@
 #include "CodeBlock.h"
 
+#include "JSValue.h"
 #include "Machine.h"
 #include "debugger.h"
-#include "JSValue.h"
 #include <stdio.h>
 
@@ -218 +218 @@
         }
         case op_new_array: {
-            int r0 = (++it)->u.operand;
-            printf("[%4d] new_array\t %s\n", location, registerName(r0).c_str());
+            int dst = (++it)->u.operand;
+            int argv = (++it)->u.operand;
+            int argc = (++it)->u.operand;
+            printf("[%4d] new_array\t %s, %s, %d\n", location, registerName(dst).c_str(), registerName(argv).c_str(), argc);
             break;
         }