Changeset 38209 in webkit for trunk/JavaScriptCore/VM/Machine.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/VM/Machine.h (modified) (3 diffs)

trunk/JavaScriptCore/VM/Machine.h

@@ -191 +191 @@
         static JSValue* SFX_CALL cti_op_mul(CTI_ARGS);
         static JSObject* SFX_CALL cti_op_new_func(CTI_ARGS);
-        static VoidPtrPair SFX_CALL cti_op_call_JSFunction(CTI_ARGS);
+        static void* SFX_CALL cti_op_call_JSFunction(CTI_ARGS);
+        static VoidPtrPair SFX_CALL cti_op_call_arityCheck(CTI_ARGS);
         static JSValue* SFX_CALL cti_op_call_NotJSFunction(CTI_ARGS);
         static void SFX_CALL cti_op_create_arguments(CTI_ARGS);
@@ -203 +204 @@
         static JSValue* SFX_CALL cti_op_resolve(CTI_ARGS);
         static JSValue* SFX_CALL cti_op_resolve_global(CTI_ARGS);
-        static JSObject* SFX_CALL cti_op_construct_JSConstructFast(CTI_ARGS);
-        static VoidPtrPair SFX_CALL cti_op_construct_JSConstruct(CTI_ARGS);
+        static JSObject* SFX_CALL cti_op_construct_JSConstruct(CTI_ARGS);
         static JSValue* SFX_CALL cti_op_construct_NotJSConstruct(CTI_ARGS);
         static JSValue* SFX_CALL cti_op_get_by_val(CTI_ARGS);
@@ -269 +269 @@
 
         static JSValue* SFX_CALL cti_vm_throw(CTI_ARGS);
-        static void* SFX_CALL cti_vm_compile(CTI_ARGS);
         static void* SFX_CALL cti_vm_lazyLinkCall(CTI_ARGS);
         static JSObject* SFX_CALL cti_op_push_activation(CTI_ARGS);