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

Timestamp:

Apr 25, 2003, 3:47:05 PM (22 years ago)

Author:

darin

Message:

Reviewed by Maciej.

• a couple improvements that give a 6.6% speedup on iBench JavaScript

• kjs/nodes.cpp: (ResolveNode::evaluate): Don't use evaluateReference.

• kjs/object.cpp: (ObjectImp::get): Do the prototype work with the ValueImp, not a wrapper. Contributes a tiny bit to the speedup, but cleaner anyway. (ObjectImp::hasProperty): Same thing here.

File:

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

trunk/JavaScriptCore/kjs/nodes.cpp

@@ -222 +222 @@
 Value ResolveNode::evaluate(ExecState *exec)
 {
-  return evaluateReference(exec).getValue(exec);
+  // This is the same as calling evaluateReference(exec).getValue(exec),
+  // only considerably faster.
+  
+  ScopeChain chain = exec->context().imp()->scopeChain();
+
+  while (!chain.isEmpty()) {
+    ObjectImp *o = chain.top();
+    Value result = o->get(exec, ident);
+    if (result.type() != UndefinedType)
+      return result;
+    chain.pop();
+  }
+
+  return Reference(Null(), ident).getValue(exec);
 }