Changeset 251529 in webkit for trunk/Source/JavaScriptCore/interpreter/CallFrame.cpp

Timestamp:

Oct 23, 2019, 10:34:21 PM (6 years ago)

Author:

[email protected]

Message:

[JSC] Remove wasmAwareLexicalGlobalObject
​https://bugs.webkit.org/show_bug.cgi?id=203351

Reviewed by Mark Lam.

Source/JavaScriptCore:

CallFrame::lexicalGlobalObject() is no longer called frequently. We can just make the current wasmAwareLexicalGlobalObject as CallFrame::lexicalGlobalObject,
and remove wasmAwareLexicalGlobalObject function.

• debugger/Debugger.cpp:

(JSC::Debugger::hasBreakpoint):
(JSC::Debugger::breakProgram):
(JSC::lexicalGlobalObjectForCallFrame):

• debugger/DebuggerCallFrame.cpp:

(JSC::DebuggerCallFrame::deprecatedVMEntryGlobalObject const):
(JSC::DebuggerCallFrame::scope):
(JSC::DebuggerCallFrame::thisValue const):
(JSC::DebuggerCallFrame::evaluateWithScopeExtension):

• debugger/DebuggerCallFrame.h:
• inspector/JSJavaScriptCallFrame.cpp:

(Inspector::JSJavaScriptCallFrame::thisObject const):

• inspector/JavaScriptCallFrame.h:

(Inspector::JavaScriptCallFrame::thisValue const):

• interpreter/CallFrame.cpp:

(JSC::CallFrame::lexicalGlobalObjectFromWasmCallee const):
(JSC::CallFrame::wasmAwareLexicalGlobalObject): Deleted.

• interpreter/CallFrame.h:
• interpreter/Interpreter.cpp:

(JSC::notifyDebuggerOfUnwinding):
(JSC::Interpreter::debug):

• interpreter/StackVisitor.cpp:

(JSC::StackVisitor::Frame::createArguments):

• interpreter/StackVisitor.h:
• llint/LLIntSlowPaths.cpp:

(JSC::LLInt::llint_throw_stack_overflow_error):

• runtime/JSFunction.cpp:

(JSC::RetrieveArgumentsFunctor::RetrieveArgumentsFunctor):
(JSC::RetrieveArgumentsFunctor::operator() const):
(JSC::retrieveArguments):

• runtime/JSScope.h:

(JSC::CallFrame::lexicalGlobalObject const):

• runtime/RegExpInlines.h:

(JSC::RegExp::matchInline):

• wasm/js/WasmToJS.cpp:

(JSC::Wasm::wasmToJS):

Source/WebCore:

• bindings/js/CommonVM.cpp:

(WebCore::lexicalFrameFromCommonVM):

• bindings/js/JSDOMWindowBase.cpp:

(WebCore::responsibleDocument):

• bindings/js/StructuredClone.cpp:

(WebCore::cloneArrayBufferImpl):

• dom/Document.cpp:

(WebCore::Document::shouldBypassMainWorldContentSecurityPolicy const):

• testing/Internals.cpp:

(WebCore::Internals::parserMetaData):
(WebCore::Internals::isFromCurrentWorld const):

File:

• trunk/Source/JavaScriptCore/interpreter/CallFrame.cpp (modified) (2 diffs)

trunk/Source/JavaScriptCore/interpreter/CallFrame.cpp

@@ -191 +191 @@
 }
 
-JSGlobalObject* CallFrame::wasmAwareLexicalGlobalObject(VM& vm)
-{
-#if ENABLE(WEBASSEMBLY)
-    if (!callee().isWasm())
-        return lexicalGlobalObject();
-    return vm.wasmContext.load()->owner<JSWebAssemblyInstance>()->globalObject();
-#else
-    UNUSED_PARAM(vm);
-    return lexicalGlobalObject();
-#endif
-}
-
 bool CallFrame::isAnyWasmCallee()
 {
@@ -358 +346 @@
 }
 
+#if ENABLE(WEBASSEMBLY)
+JSGlobalObject* CallFrame::lexicalGlobalObjectFromWasmCallee(VM& vm) const
+{
+    return vm.wasmContext.load()->owner<JSWebAssemblyInstance>()->globalObject();
+}
+#endif
+
 bool isFromJSCode(void* returnAddress)
 {