Changeset 37664 in webkit for trunk/JavaScriptCore

Timestamp:

Oct 17, 2008, 3:26:44 PM (17 years ago)

Author:

[email protected]

Message:

2008-10-17 Maciej Stachowiak <​[email protected]>

Reviewed by Cameron Zwarich

• don't bother to do arguments tearoff when it will have no effect

~1% on v8 raytrace

• VM/CodeGenerator.cpp: (JSC::CodeGenerator::emitReturn):

Location:

trunk/JavaScriptCore

Files:

• ChangeLog (modified) (1 diff)
• VM/CodeGenerator.cpp (modified) (1 diff)

trunk/JavaScriptCore/ChangeLog

@@ +1 @@
+2008-10-17  Maciej Stachowiak  <[email protected]>
+
+        Reviewed by Cameron Zwarich
+        
+        - don't bother to do arguments tearoff when it will have no effect
+
+        ~1% on v8 raytrace
+        
+        * VM/CodeGenerator.cpp:
+        (JSC::CodeGenerator::emitReturn):
+
 2008-10-17  Marco Barisione  <[email protected]>
 

trunk/JavaScriptCore/VM/CodeGenerator.cpp

@@ -1169 +1169 @@
         emitOpcode(op_tear_off_activation);
         instructions().append(m_activationRegisterIndex);
-    } else if (m_codeBlock->usesArguments)
+    } else if (m_codeBlock->usesArguments && m_codeBlock->numParameters > 1)
         emitOpcode(op_tear_off_arguments);