<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.1//EN"
"http://www.w3.org/TR/xhtml11/DTD/xhtml11.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<head><meta http-equiv="content-type" content="text/html; charset=utf-8" />
<title>[187555] trunk</title>
</head>
<body>
<style type="text/css"><!--
#msg dl.meta { border: 1px #006 solid; background: #369; padding: 6px; color: #fff; }
#msg dl.meta dt { float: left; width: 6em; font-weight: bold; }
#msg dt:after { content:':';}
#msg dl, #msg dt, #msg ul, #msg li, #header, #footer, #logmsg { font-family: verdana,arial,helvetica,sans-serif; font-size: 10pt; }
#msg dl a { font-weight: bold}
#msg dl a:link { color:#fc3; }
#msg dl a:active { color:#ff0; }
#msg dl a:visited { color:#cc6; }
h3 { font-family: verdana,arial,helvetica,sans-serif; font-size: 10pt; font-weight: bold; }
#msg pre { overflow: auto; background: #ffc; border: 1px #fa0 solid; padding: 6px; }
#logmsg { background: #ffc; border: 1px #fa0 solid; padding: 1em 1em 0 1em; }
#logmsg p, #logmsg pre, #logmsg blockquote { margin: 0 0 1em 0; }
#logmsg p, #logmsg li, #logmsg dt, #logmsg dd { line-height: 14pt; }
#logmsg h1, #logmsg h2, #logmsg h3, #logmsg h4, #logmsg h5, #logmsg h6 { margin: .5em 0; }
#logmsg h1:first-child, #logmsg h2:first-child, #logmsg h3:first-child, #logmsg h4:first-child, #logmsg h5:first-child, #logmsg h6:first-child { margin-top: 0; }
#logmsg ul, #logmsg ol { padding: 0; list-style-position: inside; margin: 0 0 0 1em; }
#logmsg ul { text-indent: -1em; padding-left: 1em; }#logmsg ol { text-indent: -1.5em; padding-left: 1.5em; }
#logmsg > ul, #logmsg > ol { margin: 0 0 1em 0; }
#logmsg pre { background: #eee; padding: 1em; }
#logmsg blockquote { border: 1px solid #fa0; border-left-width: 10px; padding: 1em 1em 0 1em; background: white;}
#logmsg dl { margin: 0; }
#logmsg dt { font-weight: bold; }
#logmsg dd { margin: 0; padding: 0 0 0.5em 0; }
#logmsg dd:before { content:'\00bb';}
#logmsg table { border-spacing: 0px; border-collapse: collapse; border-top: 4px solid #fa0; border-bottom: 1px solid #fa0; background: #fff; }
#logmsg table th { text-align: left; font-weight: normal; padding: 0.2em 0.5em; border-top: 1px dotted #fa0; }
#logmsg table td { text-align: right; border-top: 1px dotted #fa0; padding: 0.2em 0.5em; }
#logmsg table thead th { text-align: center; border-bottom: 1px solid #fa0; }
#logmsg table th.Corner { text-align: left; }
#logmsg hr { border: none 0; border-top: 2px dashed #fa0; height: 1px; }
#header, #footer { color: #fff; background: #636; border: 1px #300 solid; padding: 6px; }
#patch { width: 100%; }
#patch h4 {font-family: verdana,arial,helvetica,sans-serif;font-size:10pt;padding:8px;background:#369;color:#fff;margin:0;}
#patch .propset h4, #patch .binary h4 {margin:0;}
#patch pre {padding:0;line-height:1.2em;margin:0;}
#patch .diff {width:100%;background:#eee;padding: 0 0 10px 0;overflow:auto;}
#patch .propset .diff, #patch .binary .diff {padding:10px 0;}
#patch span {display:block;padding:0 10px;}
#patch .modfile, #patch .addfile, #patch .delfile, #patch .propset, #patch .binary, #patch .copfile {border:1px solid #ccc;margin:10px 0;}
#patch ins {background:#dfd;text-decoration:none;display:block;padding:0 10px;}
#patch del {background:#fdd;text-decoration:none;display:block;padding:0 10px;}
#patch .lines, .info {color:#888;background:#fff;}
--></style>
<div id="msg">
<dl class="meta">
<dt>Revision</dt> <dd><a href="http://trac.webkit.org/projects/webkit/changeset/187555">187555</a></dd>
<dt>Author</dt> <dd>basile_clement@apple.com</dd>
<dt>Date</dt> <dd>2015-07-29 13:30:08 -0700 (Wed, 29 Jul 2015)</dd>
</dl>
<h3>Log Message</h3>
<pre>Remove native call inlining
https://bugs.webkit.org/show_bug.cgi?id=147417
.:
Rubber-stamped by Filip Pizlo.
* Source/cmake/OptionsEfl.cmake:
* Source/cmake/WebKitFeatures.cmake:
Source/JavaScriptCore:
Rubber Stamped by Filip Pizlo.
* CMakeLists.txt:
* dfg/DFGAbstractInterpreterInlines.h:
(JSC::DFG::AbstractInterpreter<AbstractStateType>::executeEffects): Deleted.
* dfg/DFGByteCodeParser.cpp:
(JSC::DFG::ByteCodeParser::handleCall): Deleted.
* dfg/DFGClobberize.h:
(JSC::DFG::clobberize): Deleted.
* dfg/DFGDoesGC.cpp:
(JSC::DFG::doesGC): Deleted.
* dfg/DFGFixupPhase.cpp:
(JSC::DFG::FixupPhase::fixupNode): Deleted.
* dfg/DFGNode.h:
(JSC::DFG::Node::hasHeapPrediction): Deleted.
(JSC::DFG::Node::hasCellOperand): Deleted.
* dfg/DFGNodeType.h:
* dfg/DFGPredictionPropagationPhase.cpp:
(JSC::DFG::PredictionPropagationPhase::propagate): Deleted.
* dfg/DFGSafeToExecute.h:
(JSC::DFG::safeToExecute): Deleted.
* dfg/DFGSpeculativeJIT32_64.cpp:
(JSC::DFG::SpeculativeJIT::compile): Deleted.
* dfg/DFGSpeculativeJIT64.cpp:
(JSC::DFG::SpeculativeJIT::compile): Deleted.
* ftl/FTLCapabilities.cpp:
(JSC::FTL::canCompile): Deleted.
* ftl/FTLLowerDFGToLLVM.cpp:
(JSC::FTL::DFG::LowerDFGToLLVM::lower): Deleted.
(JSC::FTL::DFG::LowerDFGToLLVM::compileNode): Deleted.
(JSC::FTL::DFG::LowerDFGToLLVM::compileNativeCallOrConstruct): Deleted.
(JSC::FTL::DFG::LowerDFGToLLVM::getFunctionBySymbol): Deleted.
(JSC::FTL::DFG::LowerDFGToLLVM::getModuleByPathForSymbol): Deleted.
(JSC::FTL::DFG::LowerDFGToLLVM::didOverflowStack): Deleted.
* ftl/FTLState.cpp:
(JSC::FTL::State::State): Deleted.
* ftl/FTLState.h:
* runtime/BundlePath.cpp: Removed.
(JSC::bundlePath): Deleted.
* runtime/JSDataViewPrototype.cpp:
(JSC::getData):
(JSC::setData):
* runtime/Options.h:
Source/WTF:
Rubber-stamped by Filip Pizlo.
* wtf/Platform.h:</pre>
<h3>Modified Paths</h3>
<ul>
<li><a href="#trunkChangeLog">trunk/ChangeLog</a></li>
<li><a href="#trunkSourceJavaScriptCoreCMakeListstxt">trunk/Source/JavaScriptCore/CMakeLists.txt</a></li>
<li><a href="#trunkSourceJavaScriptCoreChangeLog">trunk/Source/JavaScriptCore/ChangeLog</a></li>
<li><a href="#trunkSourceJavaScriptCoredfgDFGAbstractInterpreterInlinesh">trunk/Source/JavaScriptCore/dfg/DFGAbstractInterpreterInlines.h</a></li>
<li><a href="#trunkSourceJavaScriptCoredfgDFGByteCodeParsercpp">trunk/Source/JavaScriptCore/dfg/DFGByteCodeParser.cpp</a></li>
<li><a href="#trunkSourceJavaScriptCoredfgDFGClobberizeh">trunk/Source/JavaScriptCore/dfg/DFGClobberize.h</a></li>
<li><a href="#trunkSourceJavaScriptCoredfgDFGDoesGCcpp">trunk/Source/JavaScriptCore/dfg/DFGDoesGC.cpp</a></li>
<li><a href="#trunkSourceJavaScriptCoredfgDFGFixupPhasecpp">trunk/Source/JavaScriptCore/dfg/DFGFixupPhase.cpp</a></li>
<li><a href="#trunkSourceJavaScriptCoredfgDFGNodeh">trunk/Source/JavaScriptCore/dfg/DFGNode.h</a></li>
<li><a href="#trunkSourceJavaScriptCoredfgDFGNodeTypeh">trunk/Source/JavaScriptCore/dfg/DFGNodeType.h</a></li>
<li><a href="#trunkSourceJavaScriptCoredfgDFGPredictionPropagationPhasecpp">trunk/Source/JavaScriptCore/dfg/DFGPredictionPropagationPhase.cpp</a></li>
<li><a href="#trunkSourceJavaScriptCoredfgDFGSafeToExecuteh">trunk/Source/JavaScriptCore/dfg/DFGSafeToExecute.h</a></li>
<li><a href="#trunkSourceJavaScriptCoredfgDFGSpeculativeJIT32_64cpp">trunk/Source/JavaScriptCore/dfg/DFGSpeculativeJIT32_64.cpp</a></li>
<li><a href="#trunkSourceJavaScriptCoredfgDFGSpeculativeJIT64cpp">trunk/Source/JavaScriptCore/dfg/DFGSpeculativeJIT64.cpp</a></li>
<li><a href="#trunkSourceJavaScriptCoreftlFTLCapabilitiescpp">trunk/Source/JavaScriptCore/ftl/FTLCapabilities.cpp</a></li>
<li><a href="#trunkSourceJavaScriptCoreftlFTLLowerDFGToLLVMcpp">trunk/Source/JavaScriptCore/ftl/FTLLowerDFGToLLVM.cpp</a></li>
<li><a href="#trunkSourceJavaScriptCoreftlFTLStatecpp">trunk/Source/JavaScriptCore/ftl/FTLState.cpp</a></li>
<li><a href="#trunkSourceJavaScriptCoreftlFTLStateh">trunk/Source/JavaScriptCore/ftl/FTLState.h</a></li>
<li><a href="#trunkSourceJavaScriptCoreruntimeJSDataViewPrototypecpp">trunk/Source/JavaScriptCore/runtime/JSDataViewPrototype.cpp</a></li>
<li><a href="#trunkSourceJavaScriptCoreruntimeOptionsh">trunk/Source/JavaScriptCore/runtime/Options.h</a></li>
<li><a href="#trunkSourceWTFChangeLog">trunk/Source/WTF/ChangeLog</a></li>
<li><a href="#trunkSourceWTFwtfPlatformh">trunk/Source/WTF/wtf/Platform.h</a></li>
<li><a href="#trunkSourcecmakeOptionsEflcmake">trunk/Source/cmake/OptionsEfl.cmake</a></li>
<li><a href="#trunkSourcecmakeWebKitFeaturescmake">trunk/Source/cmake/WebKitFeatures.cmake</a></li>
</ul>
<h3>Removed Paths</h3>
<ul>
<li><a href="#trunkSourceJavaScriptCoreruntimeBundlePathcpp">trunk/Source/JavaScriptCore/runtime/BundlePath.cpp</a></li>
</ul>
</div>
<div id="patch">
<h3>Diff</h3>
<a id="trunkChangeLog"></a>
<div class="modfile"><h4>Modified: trunk/ChangeLog (187554 => 187555)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/ChangeLog 2015-07-29 20:10:54 UTC (rev 187554)
+++ trunk/ChangeLog 2015-07-29 20:30:08 UTC (rev 187555)
</span><span class="lines">@@ -1,3 +1,13 @@
</span><ins>+2015-07-29 Basile Clement <basile_clement@apple.com>
+
+ Remove native call inlining
+ https://bugs.webkit.org/show_bug.cgi?id=147417
+
+ Rubber-stamped by Filip Pizlo.
+
+ * Source/cmake/OptionsEfl.cmake:
+ * Source/cmake/WebKitFeatures.cmake:
+
</ins><span class="cx"> 2015-07-28 Yusuke Suzuki <utatane.tea@gmail.com>
</span><span class="cx">
</span><span class="cx"> [ES6] Add ENABLE_ES6_MODULES compile time flag with the default value "false"
</span></span></pre></div>
<a id="trunkSourceJavaScriptCoreCMakeListstxt"></a>
<div class="modfile"><h4>Modified: trunk/Source/JavaScriptCore/CMakeLists.txt (187554 => 187555)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/JavaScriptCore/CMakeLists.txt 2015-07-29 20:10:54 UTC (rev 187554)
+++ trunk/Source/JavaScriptCore/CMakeLists.txt 2015-07-29 20:30:08 UTC (rev 187555)
</span><span class="lines">@@ -434,7 +434,6 @@
</span><span class="cx"> runtime/BooleanConstructor.cpp
</span><span class="cx"> runtime/BooleanObject.cpp
</span><span class="cx"> runtime/BooleanPrototype.cpp
</span><del>- runtime/BundlePath.cpp
</del><span class="cx"> runtime/CallData.cpp
</span><span class="cx"> runtime/ClonedArguments.cpp
</span><span class="cx"> runtime/CodeCache.cpp
</span><span class="lines">@@ -809,43 +808,6 @@
</span><span class="cx"> COMMAND ${CMAKE_COMMAND} -E touch ${DERIVED_SOURCES_JAVASCRIPTCORE_DIR}/WebKitLLVMLibraryToken.h
</span><span class="cx"> VERBATIM)
</span><span class="cx">
</span><del>- if (ENABLE_FTL_NATIVE_CALL_INLINING)
- function(JOIN VALUES GLUE OUTPUT)
- string(REPLACE ";" "${GLUE}" _TMP_STR "${VALUES}")
- set(${OUTPUT} "${_TMP_STR}" PARENT_SCOPE)
- endfunction()
-
- JOIN("${JavaScriptCore_INCLUDE_DIRECTORIES}" " -I" JSC_INCLUDES)
-
- set(LLVM_BITCODE_FILES)
-
- foreach (_file ${JavaScriptCore_RUNTIME_SOURCES})
- get_filename_component(_name ${_file} NAME_WE)
- add_custom_command(
- OUTPUT ${CMAKE_RUNTIME_OUTPUT_DIRECTORY}/runtime/${_name}.bc
- COMMAND ${PYTHON_EXECUTABLE} ${JAVASCRIPTCORE_DIR}/create-llvm-ir-from-source-file.py ${_file} ${CMAKE_RUNTIME_OUTPUT_DIRECTORY} ${CLANG_EXE} "${JSC_INCLUDES}"
- WORKING_DIRECTORY "${JAVASCRIPTCORE_DIR}"
- VERBATIM)
-
- ADD_SOURCE_DEPENDENCIES(${CMAKE_CURRENT_SOURCE_DIR}/ftl/FTLState.cpp ${CMAKE_RUNTIME_OUTPUT_DIRECTORY}/runtime/${_name}.bc)
- list(APPEND LLVM_BITCODE_FILES
- ${CMAKE_RUNTIME_OUTPUT_DIRECTORY}/runtime/${_name}.bc
- )
- endforeach ()
-
- get_filename_component(LLVM_BINS ${LLVM_CONFIG_EXE} PATH)
-
- add_custom_command(
- OUTPUT ${DERIVED_SOURCES_JAVASCRIPTCORE_DIR}/InlineRuntimeSymbolTable.h
- MAIN_DEPENDENCY ${JAVASCRIPTCORE_DIR}/create-symbol-table-index.py
- DEPENDS ${LLVM_BITCODE_FILES}
- COMMAND ${PYTHON_EXECUTABLE} ${JAVASCRIPTCORE_DIR}/create-symbol-table-index.py ${CMAKE_RUNTIME_OUTPUT_DIRECTORY} ${JAVASCRIPTCORE_DIR} ${DERIVED_SOURCES_JAVASCRIPTCORE_DIR} ${LLVM_BINS}
- WORKING_DIRECTORY "${JAVASCRIPTCORE_DIR}"
- VERBATIM)
-
- ADD_SOURCE_DEPENDENCIES(${CMAKE_CURRENT_SOURCE_DIR}/ftl/FTLState.cpp ${DERIVED_SOURCES_JAVASCRIPTCORE_DIR}/InlineRuntimeSymbolTable.h)
- endif ()
-
</del><span class="cx"> WEBKIT_WRAP_SOURCELIST(${llvmForJSC_SOURCES})
</span><span class="cx"> add_library(llvmForJSC SHARED ${llvmForJSC_SOURCES} ${DERIVED_SOURCES_JAVASCRIPTCORE_DIR}/WebKitLLVMLibraryToken.h)
</span><span class="cx"> target_link_libraries(llvmForJSC ${LLVM_STATIC_LIBRARIES} "pthread" "dl" -Wl,--version-script=${JAVASCRIPTCORE_DIR}/llvm/library/libllvmForJSC.version)
</span></span></pre></div>
<a id="trunkSourceJavaScriptCoreChangeLog"></a>
<div class="modfile"><h4>Modified: trunk/Source/JavaScriptCore/ChangeLog (187554 => 187555)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/JavaScriptCore/ChangeLog 2015-07-29 20:10:54 UTC (rev 187554)
+++ trunk/Source/JavaScriptCore/ChangeLog 2015-07-29 20:30:08 UTC (rev 187555)
</span><span class="lines">@@ -1,5 +1,54 @@
</span><span class="cx"> 2015-07-29 Basile Clement <basile_clement@apple.com>
</span><span class="cx">
</span><ins>+ Remove native call inlining
+ https://bugs.webkit.org/show_bug.cgi?id=147417
+
+ Rubber Stamped by Filip Pizlo.
+
+ * CMakeLists.txt:
+ * dfg/DFGAbstractInterpreterInlines.h:
+ (JSC::DFG::AbstractInterpreter<AbstractStateType>::executeEffects): Deleted.
+ * dfg/DFGByteCodeParser.cpp:
+ (JSC::DFG::ByteCodeParser::handleCall): Deleted.
+ * dfg/DFGClobberize.h:
+ (JSC::DFG::clobberize): Deleted.
+ * dfg/DFGDoesGC.cpp:
+ (JSC::DFG::doesGC): Deleted.
+ * dfg/DFGFixupPhase.cpp:
+ (JSC::DFG::FixupPhase::fixupNode): Deleted.
+ * dfg/DFGNode.h:
+ (JSC::DFG::Node::hasHeapPrediction): Deleted.
+ (JSC::DFG::Node::hasCellOperand): Deleted.
+ * dfg/DFGNodeType.h:
+ * dfg/DFGPredictionPropagationPhase.cpp:
+ (JSC::DFG::PredictionPropagationPhase::propagate): Deleted.
+ * dfg/DFGSafeToExecute.h:
+ (JSC::DFG::safeToExecute): Deleted.
+ * dfg/DFGSpeculativeJIT32_64.cpp:
+ (JSC::DFG::SpeculativeJIT::compile): Deleted.
+ * dfg/DFGSpeculativeJIT64.cpp:
+ (JSC::DFG::SpeculativeJIT::compile): Deleted.
+ * ftl/FTLCapabilities.cpp:
+ (JSC::FTL::canCompile): Deleted.
+ * ftl/FTLLowerDFGToLLVM.cpp:
+ (JSC::FTL::DFG::LowerDFGToLLVM::lower): Deleted.
+ (JSC::FTL::DFG::LowerDFGToLLVM::compileNode): Deleted.
+ (JSC::FTL::DFG::LowerDFGToLLVM::compileNativeCallOrConstruct): Deleted.
+ (JSC::FTL::DFG::LowerDFGToLLVM::getFunctionBySymbol): Deleted.
+ (JSC::FTL::DFG::LowerDFGToLLVM::getModuleByPathForSymbol): Deleted.
+ (JSC::FTL::DFG::LowerDFGToLLVM::didOverflowStack): Deleted.
+ * ftl/FTLState.cpp:
+ (JSC::FTL::State::State): Deleted.
+ * ftl/FTLState.h:
+ * runtime/BundlePath.cpp: Removed.
+ (JSC::bundlePath): Deleted.
+ * runtime/JSDataViewPrototype.cpp:
+ (JSC::getData):
+ (JSC::setData):
+ * runtime/Options.h:
+
+2015-07-29 Basile Clement <basile_clement@apple.com>
+
</ins><span class="cx"> Unreviewed, skipping a test that is too complex for its own good
</span><span class="cx"> https://bugs.webkit.org/show_bug.cgi?id=147167
</span><span class="cx">
</span></span></pre></div>
<a id="trunkSourceJavaScriptCoredfgDFGAbstractInterpreterInlinesh"></a>
<div class="modfile"><h4>Modified: trunk/Source/JavaScriptCore/dfg/DFGAbstractInterpreterInlines.h (187554 => 187555)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/JavaScriptCore/dfg/DFGAbstractInterpreterInlines.h 2015-07-29 20:10:54 UTC (rev 187554)
+++ trunk/Source/JavaScriptCore/dfg/DFGAbstractInterpreterInlines.h 2015-07-29 20:30:08 UTC (rev 187555)
</span><span class="lines">@@ -2319,8 +2319,6 @@
</span><span class="cx">
</span><span class="cx"> case Call:
</span><span class="cx"> case Construct:
</span><del>- case NativeCall:
- case NativeConstruct:
</del><span class="cx"> case CallVarargs:
</span><span class="cx"> case CallForwardVarargs:
</span><span class="cx"> case ConstructVarargs:
</span></span></pre></div>
<a id="trunkSourceJavaScriptCoredfgDFGByteCodeParsercpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/JavaScriptCore/dfg/DFGByteCodeParser.cpp (187554 => 187555)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/JavaScriptCore/dfg/DFGByteCodeParser.cpp 2015-07-29 20:10:54 UTC (rev 187554)
+++ trunk/Source/JavaScriptCore/dfg/DFGByteCodeParser.cpp 2015-07-29 20:30:08 UTC (rev 187555)
</span><span class="lines">@@ -1066,25 +1066,6 @@
</span><span class="cx"> return;
</span><span class="cx"> }
</span><span class="cx">
</span><del>-#if ENABLE(FTL_NATIVE_CALL_INLINING)
- if (isFTL(m_graph.m_plan.mode) && Options::optimizeNativeCalls() && callLinkStatus.size() == 1 && !callLinkStatus.couldTakeSlowPath()) {
- CallVariant callee = callLinkStatus[0];
- JSFunction* function = callee.function();
- CodeSpecializationKind specializationKind = InlineCallFrame::specializationKindFor(kind);
- if (function && function->isHostFunction()) {
- emitFunctionChecks(callee, callTarget, virtualRegisterForArgument(0, registerOffset));
- callOpInfo = OpInfo(m_graph.freeze(function));
-
- if (op == Call)
- op = NativeCall;
- else {
- ASSERT(op == Construct);
- op = NativeConstruct;
- }
- }
- }
-#endif
-
</del><span class="cx"> addCall(result, op, callOpInfo, callTarget, argumentCountIncludingThis, registerOffset, prediction);
</span><span class="cx"> }
</span><span class="cx">
</span></span></pre></div>
<a id="trunkSourceJavaScriptCoredfgDFGClobberizeh"></a>
<div class="modfile"><h4>Modified: trunk/Source/JavaScriptCore/dfg/DFGClobberize.h (187554 => 187555)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/JavaScriptCore/dfg/DFGClobberize.h 2015-07-29 20:10:54 UTC (rev 187554)
+++ trunk/Source/JavaScriptCore/dfg/DFGClobberize.h 2015-07-29 20:30:08 UTC (rev 187555)
</span><span class="lines">@@ -377,8 +377,6 @@
</span><span class="cx"> case ArrayPop:
</span><span class="cx"> case Call:
</span><span class="cx"> case Construct:
</span><del>- case NativeCall:
- case NativeConstruct:
</del><span class="cx"> case CallVarargs:
</span><span class="cx"> case CallForwardVarargs:
</span><span class="cx"> case ConstructVarargs:
</span></span></pre></div>
<a id="trunkSourceJavaScriptCoredfgDFGDoesGCcpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/JavaScriptCore/dfg/DFGDoesGC.cpp (187554 => 187555)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/JavaScriptCore/dfg/DFGDoesGC.cpp 2015-07-29 20:10:54 UTC (rev 187554)
+++ trunk/Source/JavaScriptCore/dfg/DFGDoesGC.cpp 2015-07-29 20:30:08 UTC (rev 187555)
</span><span class="lines">@@ -123,8 +123,6 @@
</span><span class="cx"> case LoadVarargs:
</span><span class="cx"> case CallForwardVarargs:
</span><span class="cx"> case ConstructForwardVarargs:
</span><del>- case NativeCall:
- case NativeConstruct:
</del><span class="cx"> case Breakpoint:
</span><span class="cx"> case ProfileWillCall:
</span><span class="cx"> case ProfileDidCall:
</span></span></pre></div>
<a id="trunkSourceJavaScriptCoredfgDFGFixupPhasecpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/JavaScriptCore/dfg/DFGFixupPhase.cpp (187554 => 187555)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/JavaScriptCore/dfg/DFGFixupPhase.cpp 2015-07-29 20:10:54 UTC (rev 187554)
+++ trunk/Source/JavaScriptCore/dfg/DFGFixupPhase.cpp 2015-07-29 20:30:08 UTC (rev 187555)
</span><span class="lines">@@ -1285,8 +1285,6 @@
</span><span class="cx"> case ConstructForwardVarargs:
</span><span class="cx"> case LoadVarargs:
</span><span class="cx"> case ProfileControlFlow:
</span><del>- case NativeCall:
- case NativeConstruct:
</del><span class="cx"> case NewObject:
</span><span class="cx"> case NewArrayBuffer:
</span><span class="cx"> case NewRegexp:
</span></span></pre></div>
<a id="trunkSourceJavaScriptCoredfgDFGNodeh"></a>
<div class="modfile"><h4>Modified: trunk/Source/JavaScriptCore/dfg/DFGNode.h (187554 => 187555)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/JavaScriptCore/dfg/DFGNode.h 2015-07-29 20:10:54 UTC (rev 187554)
+++ trunk/Source/JavaScriptCore/dfg/DFGNode.h 2015-07-29 20:30:08 UTC (rev 187555)
</span><span class="lines">@@ -1252,8 +1252,6 @@
</span><span class="cx"> case CallVarargs:
</span><span class="cx"> case ConstructVarargs:
</span><span class="cx"> case CallForwardVarargs:
</span><del>- case NativeCall:
- case NativeConstruct:
</del><span class="cx"> case GetByOffset:
</span><span class="cx"> case MultiGetByOffset:
</span><span class="cx"> case GetClosureVar:
</span><span class="lines">@@ -1285,8 +1283,6 @@
</span><span class="cx"> {
</span><span class="cx"> switch (op()) {
</span><span class="cx"> case CheckCell:
</span><del>- case NativeConstruct:
- case NativeCall:
</del><span class="cx"> case NewFunction:
</span><span class="cx"> case CreateActivation:
</span><span class="cx"> case MaterializeCreateActivation:
</span></span></pre></div>
<a id="trunkSourceJavaScriptCoredfgDFGNodeTypeh"></a>
<div class="modfile"><h4>Modified: trunk/Source/JavaScriptCore/dfg/DFGNodeType.h (187554 => 187555)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/JavaScriptCore/dfg/DFGNodeType.h 2015-07-29 20:10:54 UTC (rev 187554)
+++ trunk/Source/JavaScriptCore/dfg/DFGNodeType.h 2015-07-29 20:30:08 UTC (rev 187555)
</span><span class="lines">@@ -239,8 +239,6 @@
</span><span class="cx"> macro(CallForwardVarargs, NodeResultJS | NodeMustGenerate) \
</span><span class="cx"> macro(ConstructVarargs, NodeResultJS | NodeMustGenerate) \
</span><span class="cx"> macro(ConstructForwardVarargs, NodeResultJS | NodeMustGenerate) \
</span><del>- macro(NativeCall, NodeResultJS | NodeMustGenerate | NodeHasVarArgs) \
- macro(NativeConstruct, NodeResultJS | NodeMustGenerate | NodeHasVarArgs) \
</del><span class="cx"> \
</span><span class="cx"> /* Allocations. */\
</span><span class="cx"> macro(NewObject, NodeResultJS) \
</span></span></pre></div>
<a id="trunkSourceJavaScriptCoredfgDFGPredictionPropagationPhasecpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/JavaScriptCore/dfg/DFGPredictionPropagationPhase.cpp (187554 => 187555)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/JavaScriptCore/dfg/DFGPredictionPropagationPhase.cpp 2015-07-29 20:10:54 UTC (rev 187554)
+++ trunk/Source/JavaScriptCore/dfg/DFGPredictionPropagationPhase.cpp 2015-07-29 20:30:08 UTC (rev 187555)
</span><span class="lines">@@ -199,8 +199,6 @@
</span><span class="cx"> case ConstructVarargs:
</span><span class="cx"> case CallForwardVarargs:
</span><span class="cx"> case ConstructForwardVarargs:
</span><del>- case NativeCall:
- case NativeConstruct:
</del><span class="cx"> case GetGlobalVar:
</span><span class="cx"> case GetClosureVar:
</span><span class="cx"> case GetFromArguments: {
</span></span></pre></div>
<a id="trunkSourceJavaScriptCoredfgDFGSafeToExecuteh"></a>
<div class="modfile"><h4>Modified: trunk/Source/JavaScriptCore/dfg/DFGSafeToExecute.h (187554 => 187555)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/JavaScriptCore/dfg/DFGSafeToExecute.h 2015-07-29 20:10:54 UTC (rev 187554)
+++ trunk/Source/JavaScriptCore/dfg/DFGSafeToExecute.h 2015-07-29 20:30:08 UTC (rev 187555)
</span><span class="lines">@@ -292,10 +292,6 @@
</span><span class="cx"> case ForwardVarargs:
</span><span class="cx"> return true;
</span><span class="cx">
</span><del>- case NativeCall:
- case NativeConstruct:
- return false; // TODO: add a check for already checked. https://bugs.webkit.org/show_bug.cgi?id=133769
-
</del><span class="cx"> case BottomValue:
</span><span class="cx"> // If in doubt, assume that this isn't safe to execute, just because we have no way of
</span><span class="cx"> // compiling this node.
</span></span></pre></div>
<a id="trunkSourceJavaScriptCoredfgDFGSpeculativeJIT32_64cpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/JavaScriptCore/dfg/DFGSpeculativeJIT32_64.cpp (187554 => 187555)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/JavaScriptCore/dfg/DFGSpeculativeJIT32_64.cpp 2015-07-29 20:10:54 UTC (rev 187554)
+++ trunk/Source/JavaScriptCore/dfg/DFGSpeculativeJIT32_64.cpp 2015-07-29 20:30:08 UTC (rev 187555)
</span><span class="lines">@@ -4686,8 +4686,6 @@
</span><span class="cx"> case ArithIMul:
</span><span class="cx"> case MultiGetByOffset:
</span><span class="cx"> case MultiPutByOffset:
</span><del>- case NativeCall:
- case NativeConstruct:
</del><span class="cx"> case CheckBadCell:
</span><span class="cx"> case BottomValue:
</span><span class="cx"> case PhantomNewObject:
</span></span></pre></div>
<a id="trunkSourceJavaScriptCoredfgDFGSpeculativeJIT64cpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/JavaScriptCore/dfg/DFGSpeculativeJIT64.cpp (187554 => 187555)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/JavaScriptCore/dfg/DFGSpeculativeJIT64.cpp 2015-07-29 20:10:54 UTC (rev 187554)
+++ trunk/Source/JavaScriptCore/dfg/DFGSpeculativeJIT64.cpp 2015-07-29 20:30:08 UTC (rev 187555)
</span><span class="lines">@@ -4748,8 +4748,6 @@
</span><span class="cx"> break;
</span><span class="cx"> #endif // ENABLE(FTL_JIT)
</span><span class="cx">
</span><del>- case NativeCall:
- case NativeConstruct:
</del><span class="cx"> case LastNodeType:
</span><span class="cx"> case Phi:
</span><span class="cx"> case Upsilon:
</span></span></pre></div>
<a id="trunkSourceJavaScriptCoreftlFTLCapabilitiescpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/JavaScriptCore/ftl/FTLCapabilities.cpp (187554 => 187555)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/JavaScriptCore/ftl/FTLCapabilities.cpp 2015-07-29 20:10:54 UTC (rev 187554)
+++ trunk/Source/JavaScriptCore/ftl/FTLCapabilities.cpp 2015-07-29 20:30:08 UTC (rev 187555)
</span><span class="lines">@@ -131,8 +131,6 @@
</span><span class="cx"> case ConstructVarargs:
</span><span class="cx"> case ConstructForwardVarargs:
</span><span class="cx"> case LoadVarargs:
</span><del>- case NativeCall:
- case NativeConstruct:
</del><span class="cx"> case ValueToInt32:
</span><span class="cx"> case Branch:
</span><span class="cx"> case LogicalNot:
</span></span></pre></div>
<a id="trunkSourceJavaScriptCoreftlFTLLowerDFGToLLVMcpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/JavaScriptCore/ftl/FTLLowerDFGToLLVM.cpp (187554 => 187555)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/JavaScriptCore/ftl/FTLLowerDFGToLLVM.cpp 2015-07-29 20:10:54 UTC (rev 187554)
+++ trunk/Source/JavaScriptCore/ftl/FTLLowerDFGToLLVM.cpp 2015-07-29 20:30:08 UTC (rev 187555)
</span><span class="lines">@@ -56,10 +56,6 @@
</span><span class="cx"> #include <unordered_set>
</span><span class="cx"> #include <wtf/ProcessID.h>
</span><span class="cx">
</span><del>-#if ENABLE(FTL_NATIVE_CALL_INLINING)
-#include "BundlePath.h"
-#endif
-
</del><span class="cx"> namespace JSC { namespace FTL {
</span><span class="cx">
</span><span class="cx"> using namespace DFG;
</span><span class="lines">@@ -158,29 +154,6 @@
</span><span class="cx">
</span><span class="cx"> auto preOrder = m_graph.blocksInPreOrder();
</span><span class="cx">
</span><del>- int maxNumberOfArguments = -1;
- for (BasicBlock* block : preOrder) {
- for (unsigned nodeIndex = block->size(); nodeIndex--; ) {
- Node* node = block->at(nodeIndex);
- switch (node->op()) {
- case NativeCall:
- case NativeConstruct: {
- int numArgs = node->numChildren();
- if (numArgs > maxNumberOfArguments)
- maxNumberOfArguments = numArgs;
- break;
- }
- default:
- break;
- }
- }
- }
-
- if (maxNumberOfArguments >= 0) {
- m_execState = m_out.alloca(arrayType(m_out.int64, JSStack::CallFrameHeaderSize + maxNumberOfArguments));
- m_execStorage = m_out.ptrToInt(m_execState, m_out.intPtr);
- }
-
</del><span class="cx"> LValue capturedAlloca = m_out.alloca(arrayType(m_out.int64, m_graph.m_nextMachineLocal));
</span><span class="cx">
</span><span class="cx"> m_captured = m_out.add(
</span><span class="lines">@@ -757,12 +730,6 @@
</span><span class="cx"> case ForwardVarargs:
</span><span class="cx"> compileForwardVarargs();
</span><span class="cx"> break;
</span><del>-#if ENABLE(FTL_NATIVE_CALL_INLINING)
- case NativeCall:
- case NativeConstruct:
- compileNativeCallOrConstruct();
- break;
-#endif
</del><span class="cx"> case Jump:
</span><span class="cx"> compileJump();
</span><span class="cx"> break;
</span><span class="lines">@@ -4271,54 +4238,7 @@
</span><span class="cx"> {
</span><span class="cx"> setBoolean(m_out.bitNot(boolify(m_node->child1())));
</span><span class="cx"> }
</span><del>-#if ENABLE(FTL_NATIVE_CALL_INLINING)
- void compileNativeCallOrConstruct()
- {
- int numPassedArgs = m_node->numChildren() - 1;
- int numArgs = numPassedArgs;
</del><span class="cx">
</span><del>- JSFunction* knownFunction = m_node->castOperand<JSFunction*>();
- NativeFunction function = knownFunction->nativeFunction();
-
- Dl_info info;
- if (!dladdr((void*)function, &info))
- ASSERT(false); // if we couldn't find the native function this doesn't bode well.
-
- LValue callee = getFunctionBySymbol(info.dli_sname);
-
- bool notInlinable;
- if ((notInlinable = !callee))
- callee = m_out.operation(function);
-
- m_out.storePtr(m_callFrame, m_execStorage, m_heaps.CallFrame_callerFrame);
- m_out.storePtr(constNull(m_out.intPtr), addressFor(m_execStorage, JSStack::CodeBlock));
- m_out.storePtr(weakPointer(knownFunction), addressFor(m_execStorage, JSStack::Callee));
-
- m_out.store64(m_out.constInt64(numArgs), addressFor(m_execStorage, JSStack::ArgumentCount));
-
- for (int i = 0; i < numPassedArgs; ++i) {
- m_out.storePtr(lowJSValue(m_graph.varArgChild(m_node, 1 + i)),
- addressFor(m_execStorage, JSStack::ThisArgument, i * sizeof(Register)));
- }
-
- LValue calleeCallFrame = m_out.address(m_execState, m_heaps.CallFrame_callerFrame).value();
- m_out.storePtr(m_out.ptrToInt(calleeCallFrame, m_out.intPtr), m_out.absolute(&vm().topCallFrame));
-
- LType typeCalleeArg;
- getParamTypes(getElementType(typeOf(callee)), &typeCalleeArg);
-
- LValue argument = notInlinable
- ? m_out.ptrToInt(calleeCallFrame, typeCalleeArg)
- : m_out.bitCast(calleeCallFrame, typeCalleeArg);
- LValue call = vmCall(callee, argument);
-
- if (verboseCompilationEnabled())
- dataLog("Native calling: ", info.dli_sname, "\n");
-
- setJSValue(call);
- }
-#endif
-
</del><span class="cx"> void compileCallOrConstruct()
</span><span class="cx"> {
</span><span class="cx"> int numPassedArgs = m_node->numChildren() - 1;
</span><span class="lines">@@ -5448,123 +5368,6 @@
</span><span class="cx"> setJSValue(activation);
</span><span class="cx"> }
</span><span class="cx">
</span><del>-#if ENABLE(FTL_NATIVE_CALL_INLINING)
- LValue getFunctionBySymbol(const CString symbol)
- {
- if (!m_ftlState.symbolTable.contains(symbol))
- return nullptr;
- if (!getModuleByPathForSymbol(m_ftlState.symbolTable.get(symbol), symbol))
- return nullptr;
- return getNamedFunction(m_ftlState.module, symbol.data());
- }
-
- bool getModuleByPathForSymbol(const CString path, const CString symbol)
- {
- if (m_ftlState.nativeLoadedLibraries.contains(path)) {
- LValue function = getNamedFunction(m_ftlState.module, symbol.data());
- if (!isInlinableSize(function)) {
- // We had no choice but to compile this function, but don't try to inline it ever again.
- m_ftlState.symbolTable.remove(symbol);
- return false;
- }
- return true;
- }
-
- LMemoryBuffer memBuf;
-
- ASSERT(isX86() || isARM64());
-
-#if PLATFORM(EFL)
- const CString actualPath = toCString(bundlePath().data(), "/runtime/", path.data());
-#else
- const CString actualPath = toCString(bundlePath().data(),
- isX86() ? "/Resources/Runtime/x86_64/" : "/Resources/Runtime/arm64/",
- path.data());
-#endif
-
- char* outMsg;
-
- if (createMemoryBufferWithContentsOfFile(actualPath.data(), &memBuf, &outMsg)) {
- if (Options::verboseFTLFailure())
- dataLog("Failed to load module at ", actualPath, "\n for symbol ", symbol, "\nERROR: ", outMsg, "\n");
- disposeMessage(outMsg);
- return false;
- }
-
- LModule module;
-
- if (parseBitcodeInContext(m_ftlState.context, memBuf, &module, &outMsg)) {
- if (Options::verboseFTLFailure())
- dataLog("Failed to parse module at ", actualPath, "\n for symbol ", symbol, "\nERROR: ", outMsg, "\n");
- disposeMemoryBuffer(memBuf);
- disposeMessage(outMsg);
- return false;
- }
-
- disposeMemoryBuffer(memBuf);
-
- if (LValue function = getNamedFunction(m_ftlState.module, symbol.data())) {
- if (!isInlinableSize(function)) {
- m_ftlState.symbolTable.remove(symbol);
- disposeModule(module);
- return false;
- }
- }
-
- Vector<CString> namedFunctions;
- for (LValue function = getFirstFunction(module); function; function = getNextFunction(function)) {
- CString functionName(getValueName(function));
- namedFunctions.append(functionName);
-
- for (LBasicBlock basicBlock = getFirstBasicBlock(function); basicBlock; basicBlock = getNextBasicBlock(basicBlock)) {
- for (LValue instruction = getFirstInstruction(basicBlock); instruction; instruction = getNextInstruction(instruction)) {
- setMetadata(instruction, m_tbaaKind, nullptr);
- setMetadata(instruction, m_tbaaStructKind, nullptr);
- }
- }
- }
-
- Vector<CString> namedGlobals;
- for (LValue global = getFirstGlobal(module); global; global = getNextGlobal(global)) {
- CString globalName(getValueName(global));
- namedGlobals.append(globalName);
- }
-
- if (linkModules(m_ftlState.module, module, LLVMLinkerDestroySource, &outMsg)) {
- if (Options::verboseFTLFailure())
- dataLog("Failed to link module at ", actualPath, "\n for symbol ", symbol, "\nERROR: ", outMsg, "\n");
- disposeMessage(outMsg);
- return false;
- }
-
- for (CString* symbol = namedFunctions.begin(); symbol != namedFunctions.end(); ++symbol) {
- LValue function = getNamedFunction(m_ftlState.module, symbol->data());
- LLVMLinkage linkage = getLinkage(function);
- if (linkage != LLVMInternalLinkage && linkage != LLVMPrivateLinkage)
- setVisibility(function, LLVMHiddenVisibility);
- if (!isDeclaration(function)) {
- setLinkage(function, LLVMPrivateLinkage);
- setLinkage(function, LLVMAvailableExternallyLinkage);
-
- if (ASSERT_DISABLED)
- removeFunctionAttr(function, LLVMStackProtectAttribute);
- }
- }
-
- for (CString* symbol = namedGlobals.begin(); symbol != namedGlobals.end(); ++symbol) {
- LValue global = getNamedGlobal(m_ftlState.module, symbol->data());
- LLVMLinkage linkage = getLinkage(global);
- if (linkage != LLVMInternalLinkage && linkage != LLVMPrivateLinkage)
- setVisibility(global, LLVMHiddenVisibility);
- if (!isDeclaration(global))
- setLinkage(global, LLVMPrivateLinkage);
- }
-
- m_ftlState.nativeLoadedLibraries.add(path);
- return true;
- }
-#endif
-
</del><span class="cx"> bool isInlinableSize(LValue function)
</span><span class="cx"> {
</span><span class="cx"> size_t instructionCount = 0;
</span><span class="lines">@@ -5600,8 +5403,6 @@
</span><span class="cx"> case PutById:
</span><span class="cx"> case Call:
</span><span class="cx"> case Construct:
</span><del>- case NativeCall:
- case NativeConstruct:
</del><span class="cx"> return m_out.below(
</span><span class="cx"> m_callFrame,
</span><span class="cx"> m_out.loadPtr(
</span><span class="lines">@@ -8658,8 +8459,6 @@
</span><span class="cx"> LBasicBlock m_handleExceptions;
</span><span class="cx"> HashMap<BasicBlock*, LBasicBlock> m_blocks;
</span><span class="cx">
</span><del>- LValue m_execState;
- LValue m_execStorage;
</del><span class="cx"> LValue m_callFrame;
</span><span class="cx"> LValue m_captured;
</span><span class="cx"> LValue m_tagTypeNumber;
</span></span></pre></div>
<a id="trunkSourceJavaScriptCoreftlFTLStatecpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/JavaScriptCore/ftl/FTLState.cpp (187554 => 187555)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/JavaScriptCore/ftl/FTLState.cpp 2015-07-29 20:10:54 UTC (rev 187554)
+++ trunk/Source/JavaScriptCore/ftl/FTLState.cpp 2015-07-29 20:30:08 UTC (rev 187555)
</span><span class="lines">@@ -35,10 +35,6 @@
</span><span class="cx"> #include <llvm/InitializeLLVM.h>
</span><span class="cx"> #include <stdio.h>
</span><span class="cx">
</span><del>-#if ENABLE(FTL_NATIVE_CALL_INLINING)
-#include "InlineRuntimeSymbolTable.h"
-#endif
-
</del><span class="cx"> namespace JSC { namespace FTL {
</span><span class="cx">
</span><span class="cx"> using namespace DFG;
</span><span class="lines">@@ -56,14 +52,6 @@
</span><span class="cx"> , unwindDataSection(0)
</span><span class="cx"> , unwindDataSectionSize(0)
</span><span class="cx"> {
</span><del>-
-#if ENABLE(FTL_NATIVE_CALL_INLINING)
-#define SYMBOL_TABLE_ADD(symbol, file) \
- symbolTable.fastAdd(symbol, file);
- FOR_EACH_LIBRARY_SYMBOL(SYMBOL_TABLE_ADD)
-#undef SYMBOL_TABLE_ADD
-#endif
-
</del><span class="cx"> switch (graph.m_plan.mode) {
</span><span class="cx"> case FTLMode: {
</span><span class="cx"> jitCode = adoptRef(new JITCode());
</span></span></pre></div>
<a id="trunkSourceJavaScriptCoreftlFTLStateh"></a>
<div class="modfile"><h4>Modified: trunk/Source/JavaScriptCore/ftl/FTLState.h (187554 => 187555)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/JavaScriptCore/ftl/FTLState.h 2015-07-29 20:10:54 UTC (rev 187554)
+++ trunk/Source/JavaScriptCore/ftl/FTLState.h 2015-07-29 20:30:08 UTC (rev 187555)
</span><span class="lines">@@ -89,10 +89,6 @@
</span><span class="cx"> void dumpState(LModule, const char* when);
</span><span class="cx">
</span><span class="cx"> HashSet<CString> nativeLoadedLibraries;
</span><del>-
-#if ENABLE(FTL_NATIVE_CALL_INLINING)
- HashMap<CString, CString> symbolTable;
-#endif
</del><span class="cx"> };
</span><span class="cx">
</span><span class="cx"> } } // namespace JSC::FTL
</span></span></pre></div>
<a id="trunkSourceJavaScriptCoreruntimeBundlePathcpp"></a>
<div class="delfile"><h4>Deleted: trunk/Source/JavaScriptCore/runtime/BundlePath.cpp (187554 => 187555)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/JavaScriptCore/runtime/BundlePath.cpp 2015-07-29 20:10:54 UTC (rev 187554)
+++ trunk/Source/JavaScriptCore/runtime/BundlePath.cpp 2015-07-29 20:30:08 UTC (rev 187555)
</span><span class="lines">@@ -1,52 +0,0 @@
</span><del>-/*
- * Copyright (C) 2014 Samsung Electronics
- * Copyright (C) 2014 University of Szeged
- *
- * Redistribution and use in source and binary forms, with or without
- * modification, are permitted provided that the following conditions
- * are met:
- * 1. Redistributions of source code must retain the above copyright
- * notice, this list of conditions and the following disclaimer.
- * 2. Redistributions in binary form must reproduce the above copyright
- * notice, this list of conditions and the following disclaimer in the
- * documentation and/or other materials provided with the distribution.
- *
- * THIS SOFTWARE IS PROVIDED BY APPLE INC. ``AS IS'' AND ANY
- * EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
- * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR
- * PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL APPLE INC. OR
- * CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL,
- * EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO,
- * PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR
- * PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY
- * OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
- * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
- * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
- */
-
-#include "config.h"
-#include "BundlePath.h"
-
-#if ENABLE(FTL_NATIVE_CALL_INLINING)
-
-#include <execinfo.h>
-#include <string>
-
-
-namespace JSC {
-
-const CString* constantBundlePath = nullptr;
-
-const CString& bundlePath()
-{
- if (!constantBundlePath) {
- std::string programPath(realpath(program_invocation_name, 0));
- constantBundlePath = new CString(programPath.substr(0, programPath.find_last_of("\\/")).c_str());
- }
-
- return *constantBundlePath;
-}
-
-} // namespace JSC
-
-#endif // FTL_NATIVE_CALL_INLINING
</del></span></pre></div>
<a id="trunkSourceJavaScriptCoreruntimeJSDataViewPrototypecpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/JavaScriptCore/runtime/JSDataViewPrototype.cpp (187554 => 187555)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/JavaScriptCore/runtime/JSDataViewPrototype.cpp 2015-07-29 20:10:54 UTC (rev 187554)
+++ trunk/Source/JavaScriptCore/runtime/JSDataViewPrototype.cpp 2015-07-29 20:30:08 UTC (rev 187555)
</span><span class="lines">@@ -115,7 +115,7 @@
</span><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> template<typename Adaptor>
</span><del>-EncodedJSValue ATTR_USED getData(ExecState* exec)
</del><ins>+EncodedJSValue getData(ExecState* exec)
</ins><span class="cx"> {
</span><span class="cx"> JSDataView* dataView = jsDynamicCast<JSDataView*>(exec->thisValue());
</span><span class="cx"> if (!dataView)
</span><span class="lines">@@ -160,7 +160,7 @@
</span><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> template<typename Adaptor>
</span><del>-EncodedJSValue ATTR_USED setData(ExecState* exec)
</del><ins>+EncodedJSValue setData(ExecState* exec)
</ins><span class="cx"> {
</span><span class="cx"> JSDataView* dataView = jsDynamicCast<JSDataView*>(exec->thisValue());
</span><span class="cx"> if (!dataView)
</span></span></pre></div>
<a id="trunkSourceJavaScriptCoreruntimeOptionsh"></a>
<div class="modfile"><h4>Modified: trunk/Source/JavaScriptCore/runtime/Options.h (187554 => 187555)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/JavaScriptCore/runtime/Options.h 2015-07-29 20:10:54 UTC (rev 187554)
+++ trunk/Source/JavaScriptCore/runtime/Options.h 2015-07-29 20:30:08 UTC (rev 187555)
</span><span class="lines">@@ -188,7 +188,6 @@
</span><span class="cx"> v(unsigned, maxPolymorphicCallVariantsForInlining, 5, nullptr) \
</span><span class="cx"> v(unsigned, frequentCallThreshold, 2, nullptr) \
</span><span class="cx"> v(double, minimumCallToKnownRate, 0.51, nullptr) \
</span><del>- v(bool, optimizeNativeCalls, false, nullptr) \
</del><span class="cx"> v(bool, enableMovHintRemoval, true, nullptr) \
</span><span class="cx"> v(bool, enableObjectAllocationSinking, true, nullptr) \
</span><span class="cx"> \
</span></span></pre></div>
<a id="trunkSourceWTFChangeLog"></a>
<div class="modfile"><h4>Modified: trunk/Source/WTF/ChangeLog (187554 => 187555)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WTF/ChangeLog 2015-07-29 20:10:54 UTC (rev 187554)
+++ trunk/Source/WTF/ChangeLog 2015-07-29 20:30:08 UTC (rev 187555)
</span><span class="lines">@@ -1,3 +1,12 @@
</span><ins>+2015-07-29 Basile Clement <basile_clement@apple.com>
+
+ Remove native call inlining
+ https://bugs.webkit.org/show_bug.cgi?id=147417
+
+ Rubber-stamped by Filip Pizlo.
+
+ * wtf/Platform.h:
+
</ins><span class="cx"> 2015-07-28 Yusuke Suzuki <utatane.tea@gmail.com>
</span><span class="cx">
</span><span class="cx"> [ES6] Add ENABLE_ES6_MODULES compile time flag with the default value "false"
</span></span></pre></div>
<a id="trunkSourceWTFwtfPlatformh"></a>
<div class="modfile"><h4>Modified: trunk/Source/WTF/wtf/Platform.h (187554 => 187555)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WTF/wtf/Platform.h 2015-07-29 20:10:54 UTC (rev 187554)
+++ trunk/Source/WTF/wtf/Platform.h 2015-07-29 20:30:08 UTC (rev 187555)
</span><span class="lines">@@ -784,17 +784,6 @@
</span><span class="cx"> #define ENABLE_FTL_JIT 0
</span><span class="cx"> #endif
</span><span class="cx">
</span><del>-#if !defined(ENABLE_FTL_NATIVE_CALL_INLINING)
-#define ENABLE_FTL_NATIVE_CALL_INLINING 0
-#endif
-
-/* Used to make GCC's optimization not throw away a symbol that we would need for native inlining */
-#if ENABLE(FTL_NATIVE_CALL_INLINING) && COMPILER(GCC) && !COMPILER(CLANG)
-#define ATTR_USED __attribute__ ((used))
-#else
-#define ATTR_USED
-#endif
-
</del><span class="cx"> /* Generational collector for JSC */
</span><span class="cx"> #if !defined(ENABLE_GGC)
</span><span class="cx"> #if CPU(X86_64) || CPU(X86) || CPU(ARM64) || CPU(ARM)
</span></span></pre></div>
<a id="trunkSourcecmakeOptionsEflcmake"></a>
<div class="modfile"><h4>Modified: trunk/Source/cmake/OptionsEfl.cmake (187554 => 187555)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/cmake/OptionsEfl.cmake 2015-07-29 20:10:54 UTC (rev 187554)
+++ trunk/Source/cmake/OptionsEfl.cmake 2015-07-29 20:30:08 UTC (rev 187555)
</span><span class="lines">@@ -282,9 +282,6 @@
</span><span class="cx"> message(FATAL_ERROR "FTL JIT is only available on X86_64 and AArch64 architectures.")
</span><span class="cx"> endif ()
</span><span class="cx"> SET_AND_EXPOSE_TO_BUILD(HAVE_LLVM TRUE)
</span><del>- if (ENABLE_FTL_NATIVE_CALL_INLINING)
- find_package(Clang REQUIRED)
- endif ()
</del><span class="cx"> endif ()
</span><span class="cx">
</span><span class="cx"> if (ENABLE_SUBTLE_CRYPTO)
</span></span></pre></div>
<a id="trunkSourcecmakeWebKitFeaturescmake"></a>
<div class="modfile"><h4>Modified: trunk/Source/cmake/WebKitFeatures.cmake (187554 => 187555)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/cmake/WebKitFeatures.cmake 2015-07-29 20:10:54 UTC (rev 187554)
+++ trunk/Source/cmake/WebKitFeatures.cmake 2015-07-29 20:30:08 UTC (rev 187555)
</span><span class="lines">@@ -80,7 +80,6 @@
</span><span class="cx"> WEBKIT_OPTION_DEFINE(ENABLE_FONT_LOAD_EVENTS "Toggle Font Load Events support" PRIVATE OFF)
</span><span class="cx"> WEBKIT_OPTION_DEFINE(ENABLE_FTPDIR "Toggle FTP directory support" PRIVATE ON)
</span><span class="cx"> WEBKIT_OPTION_DEFINE(ENABLE_FTL_JIT "Toggle FTL support for JSC" PRIVATE OFF)
</span><del>- WEBKIT_OPTION_DEFINE(ENABLE_FTL_NATIVE_CALL_INLINING "Toggle native call inlining for FTL" PRIVATE OFF)
</del><span class="cx"> WEBKIT_OPTION_DEFINE(ENABLE_FULLSCREEN_API "Toggle Fullscreen API support" PRIVATE OFF)
</span><span class="cx"> WEBKIT_OPTION_DEFINE(ENABLE_GAMEPAD "Toggle Gamepad support" PRIVATE OFF)
</span><span class="cx"> WEBKIT_OPTION_DEFINE(ENABLE_GAMEPAD_DEPRECATED "Toggle deprecated Gamepad support" PRIVATE OFF)
</span><span class="lines">@@ -175,7 +174,6 @@
</span><span class="cx">
</span><span class="cx"> WEBKIT_OPTION_DEPEND(ENABLE_ENCRYPTED_MEDIA_V2 ENABLE_VIDEO)
</span><span class="cx"> WEBKIT_OPTION_DEPEND(ENABLE_DFG_JIT ENABLE_JIT)
</span><del>- WEBKIT_OPTION_DEPEND(ENABLE_FTL_NATIVE_CALL_INLINING ENABLE_FTL_JIT)
</del><span class="cx"> WEBKIT_OPTION_DEPEND(ENABLE_FTL_JIT ENABLE_DFG_JIT)
</span><span class="cx"> WEBKIT_OPTION_DEPEND(ENABLE_MEDIA_CONTROLS_SCRIPT ENABLE_VIDEO)
</span><span class="cx"> WEBKIT_OPTION_DEPEND(ENABLE_VIDEO_TRACK ENABLE_VIDEO)
</span></span></pre>
</div>
</div>
</body>
</html>