Paste number 60567: Inefficient patch that fixes olliej's testcase

Paste number 60567: Inefficient patch that fixes olliej's testcase
Pasted by: cpst
5 days, 22 hours ago
#webkit | Context in IRC logs
Paste contents:
Raw Source | XML | Display As
Index: kjs/nodes.cpp
===================================================================
--- kjs/nodes.cpp        (revision 33029)
+++ kjs/nodes.cpp        (working copy)
@@ -4344,7 +4344,8 @@ found:
 
 RegisterID* AssignDotNode::emitCode(CodeGenerator& generator, RegisterID* dst)
 {
-    RefPtr<RegisterID> base = generator.emitNode(m_base.get());
+    RefPtr<RegisterID> base = generator.newTemporary();
+    generator.emitNode(base.get(), m_base.get());
     RegisterID* value = generator.emitNode(dst, m_right.get());
     return generator.emitPutById(base.get(), m_ident, value);
 }
@@ -4424,7 +4425,8 @@ JSValue* AssignErrorNode::evaluate(OldIn
 
 RegisterID* AssignBracketNode::emitCode(CodeGenerator& generator, RegisterID* dst)
 {
-    RefPtr<RegisterID> base = generator.emitNode(m_base.get());
+    RefPtr<RegisterID> base = generator.newTemporary();
+    generator.emitNode(base.get(), m_base.get());
     RefPtr<RegisterID> property = generator.emitNode(m_subscript.get());
     RegisterID* value = generator.emitNode(dst, m_right.get());
     return generator.emitPutByVal(base.get(), property.get(), value);

This paste has no annotations.

Colorize as:
Show Line Numbers

Ads absolutely not by Google

Lisppaste pastes can be made by anyone at any time. Imagine a fearsomely comprehensive disclaimer of liability. Now fear, comprehensively.