Changeset 209433 in webkit for trunk/Source/JavaScriptCore/jit/Repatch.cpp

Timestamp:

Dec 6, 2016, 4:27:58 PM (9 years ago)

Author:

[email protected]

Message:

Remove old Wasm object model
​https://bugs.webkit.org/show_bug.cgi?id=165481

Reviewed by Keith Miller and Mark Lam.

Source/JavaScriptCore:

It's confusing to see code that consults both the old
Wasm object model alongside the new one. The old object
model is not a thing, and it's not being used. Let's
remove it now to prevent further confusion.

• CMakeLists.txt:
• JavaScriptCore.xcodeproj/project.pbxproj:
• bytecode/CodeBlock.cpp:

(JSC::CodeBlock::finalizeLLIntInlineCaches):
(JSC::CodeBlock::replacement):
(JSC::CodeBlock::computeCapabilityLevel):
(JSC::CodeBlock::updateAllPredictions):

• bytecode/CodeBlock.h:
• bytecode/WebAssemblyCodeBlock.cpp: Removed.
• bytecode/WebAssemblyCodeBlock.h: Removed.
• dfg/DFGCapabilities.cpp:

(JSC::DFG::isSupportedForInlining):

• interpreter/Interpreter.cpp:

(JSC::GetStackTraceFunctor::operator()):
(JSC::UnwindFunctor::operator()):
(JSC::isWebAssemblyExecutable): Deleted.

• jit/JITOperations.cpp:
• jit/Repatch.cpp:

(JSC::linkPolymorphicCall):

• llint/LLIntSlowPaths.cpp:

(JSC::LLInt::setUpCall):

• runtime/ExecutableBase.cpp:

(JSC::ExecutableBase::clearCode):

• runtime/ExecutableBase.h:

(JSC::ExecutableBase::isWebAssemblyExecutable): Deleted.

• runtime/JSFunction.cpp:
• runtime/JSFunction.h:
• runtime/JSFunctionInlines.h:

(JSC::JSFunction::isBuiltinFunction):

• runtime/VM.cpp:

(JSC::VM::VM):

• runtime/VM.h:
• runtime/WebAssemblyExecutable.cpp: Removed.
• runtime/WebAssemblyExecutable.h: Removed.

Source/WebCore:

• testing/Internals.cpp:

(WebCore::Internals::parserMetaData):

File:

• trunk/Source/JavaScriptCore/jit/Repatch.cpp (modified) (2 diffs)

trunk/Source/JavaScriptCore/jit/Repatch.cpp

@@ -54 +54 @@
 #include "StructureStubInfo.h"
 #include "ThunkGenerators.h"
-#include "WebAssemblyCodeBlock.h"
 #include <wtf/CommaPrinter.h>
 #include <wtf/ListDump.h>
@@ -715 +714 @@
         else {
             ExecutableBase* executable = variant.executable();
-#if ENABLE(WEBASSEMBLY)
-            if (executable->isWebAssemblyExecutable())
-                codeBlock = jsCast<WebAssemblyExecutable*>(executable)->codeBlockForCall();
-            else
-#endif
-                codeBlock = jsCast<FunctionExecutable*>(executable)->codeBlockForCall();
+            codeBlock = jsCast<FunctionExecutable*>(executable)->codeBlockForCall();
             // If we cannot handle a callee, either because we don't have a CodeBlock or because arity mismatch,
             // assume that it's better for this whole thing to be a virtual call.