Changeset 163755 in webkit for trunk/Source/JavaScriptCore/dfg/DFGOperations.cpp

Timestamp:

Feb 9, 2014, 1:33:17 PM (12 years ago)

Author:

[email protected]

Message:

Pass VM instead of ExecState to JSCell::fastGetOwnProperty().
<​https://webkit.org/b/128497>

Knocks off a couple of instructions.

Reviewed by Anders Carlsson.

• dfg/DFGOperations.cpp:
• jit/JITOperations.cpp:

(JSC::getByVal):

• llint/LLIntSlowPaths.cpp:

(JSC::LLInt::getByVal):

• runtime/JSCell.h:
• runtime/JSCellInlines.h:

(JSC::JSCell::fastGetOwnProperty):

File:

• trunk/Source/JavaScriptCore/dfg/DFGOperations.cpp (modified) (4 diffs)

trunk/Source/JavaScriptCore/dfg/DFGOperations.cpp

@@ -279 +279 @@
 EncodedJSValue JIT_OPERATION operationGetByVal(ExecState* exec, EncodedJSValue encodedBase, EncodedJSValue encodedProperty)
 {
-    VM* vm = &exec->vm();
-    NativeCallFrameTracer tracer(vm, exec);
+    VM& vm = exec->vm();
+    NativeCallFrameTracer tracer(&vm, exec);
     
     JSValue baseValue = JSValue::decode(encodedBase);
@@ -296 +296 @@
                 return getByVal(exec, base, propertyAsUInt32);
         } else if (property.isString()) {
-            if (JSValue result = base->fastGetOwnProperty(exec, asString(property)->value(exec)))
+            if (JSValue result = base->fastGetOwnProperty(vm, asString(property)->value(exec)))
                 return JSValue::encode(result);
         }
@@ -310 +310 @@
 EncodedJSValue JIT_OPERATION operationGetByValCell(ExecState* exec, JSCell* base, EncodedJSValue encodedProperty)
 {
-    VM* vm = &exec->vm();
-    NativeCallFrameTracer tracer(vm, exec);
+    VM& vm = exec->vm();
+    NativeCallFrameTracer tracer(&vm, exec);
     
     JSValue property = JSValue::decode(encodedProperty);
@@ -323 +323 @@
             return getByVal(exec, base, propertyAsUInt32);
     } else if (property.isString()) {
-        if (JSValue result = base->fastGetOwnProperty(exec, asString(property)->value(exec)))
+        if (JSValue result = base->fastGetOwnProperty(vm, asString(property)->value(exec)))
             return JSValue::encode(result);
     }