Changeset 38209 in webkit for trunk/JavaScriptCore/parser/Nodes.h

Timestamp:

Nov 6, 2008, 5:30:03 PM (17 years ago)

Author:

[email protected]

Message:

2008-11-06 Gavin Barraclough ​[email protected]

Reviewed by Oliver Hunt.

Do not make a cti_* call to perform an op_call unless either:
(1) The codeblock for the function body has not been generated.
(2) The number of arguments passed does not match the callee arity.

~1% progression on sunspider --v8

• VM/CTI.cpp: (JSC::CTI::compileOpCallInitializeCallFrame): (JSC::CTI::compileOpCall): (JSC::CTI::privateCompileSlowCases):
• VM/CTI.h:
• VM/Machine.cpp: (JSC::Machine::cti_op_call_JSFunction): (JSC::Machine::cti_op_call_arityCheck): (JSC::Machine::cti_op_construct_JSConstruct):
• VM/Machine.h:
• kjs/nodes.h:

File:

• trunk/JavaScriptCore/parser/Nodes.h (modified) (1 diff)

trunk/JavaScriptCore/parser/Nodes.h

@@ -2246 +2246 @@
 
     class FunctionBodyNode : public ScopeNode {
+        friend class CTI;
     public:
         static FunctionBodyNode* create(JSGlobalData*, SourceElements*, VarStack*, FunctionStack*, const SourceCode&, CodeFeatures, int numConstants) JSC_FAST_CALL;