<!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>[212717] 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/212717">212717</a></dd>
<dt>Author</dt> <dd>ryanhaddad@apple.com</dd>
<dt>Date</dt> <dd>2017-02-21 10:00:43 -0800 (Tue, 21 Feb 2017)</dd>
</dl>
<h3>Log Message</h3>
<pre>Unreviewed, rolling out <a href="http://trac.webkit.org/projects/webkit/changeset/212712">r212712</a>.
This change broke the CLoop build.
Reverted changeset:
"JSModuleNamespace object should have IC"
https://bugs.webkit.org/show_bug.cgi?id=160590
http://trac.webkit.org/changeset/212712</pre>
<h3>Modified Paths</h3>
<ul>
<li><a href="#trunkJSTestsChangeLog">trunk/JSTests/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="#trunkSourceJavaScriptCoreJavaScriptCorexcodeprojprojectpbxproj">trunk/Source/JavaScriptCore/JavaScriptCore.xcodeproj/project.pbxproj</a></li>
<li><a href="#trunkSourceJavaScriptCorebytecodeAccessCasecpp">trunk/Source/JavaScriptCore/bytecode/AccessCase.cpp</a></li>
<li><a href="#trunkSourceJavaScriptCorebytecodeAccessCaseh">trunk/Source/JavaScriptCore/bytecode/AccessCase.h</a></li>
<li><a href="#trunkSourceJavaScriptCorebytecodeGetByIdStatuscpp">trunk/Source/JavaScriptCore/bytecode/GetByIdStatus.cpp</a></li>
<li><a href="#trunkSourceJavaScriptCorebytecodeGetByIdStatush">trunk/Source/JavaScriptCore/bytecode/GetByIdStatus.h</a></li>
<li><a href="#trunkSourceJavaScriptCorebytecodePolymorphicAccesscpp">trunk/Source/JavaScriptCore/bytecode/PolymorphicAccess.cpp</a></li>
<li><a href="#trunkSourceJavaScriptCoredfgDFGByteCodeParsercpp">trunk/Source/JavaScriptCore/dfg/DFGByteCodeParser.cpp</a></li>
<li><a href="#trunkSourceJavaScriptCorejitAssemblyHelpersh">trunk/Source/JavaScriptCore/jit/AssemblyHelpers.h</a></li>
<li><a href="#trunkSourceJavaScriptCorejitRepatchcpp">trunk/Source/JavaScriptCore/jit/Repatch.cpp</a></li>
<li><a href="#trunkSourceJavaScriptCoreruntimeAbstractModuleRecordcpp">trunk/Source/JavaScriptCore/runtime/AbstractModuleRecord.cpp</a></li>
<li><a href="#trunkSourceJavaScriptCoreruntimeJSModuleNamespaceObjectcpp">trunk/Source/JavaScriptCore/runtime/JSModuleNamespaceObject.cpp</a></li>
<li><a href="#trunkSourceJavaScriptCoreruntimeJSModuleNamespaceObjecth">trunk/Source/JavaScriptCore/runtime/JSModuleNamespaceObject.h</a></li>
<li><a href="#trunkSourceJavaScriptCoreruntimeJSModuleRecordh">trunk/Source/JavaScriptCore/runtime/JSModuleRecord.h</a></li>
<li><a href="#trunkSourceJavaScriptCoreruntimePropertySloth">trunk/Source/JavaScriptCore/runtime/PropertySlot.h</a></li>
</ul>
<h3>Removed Paths</h3>
<ul>
<li><a href="#trunkJSTestsmodulesmoduleassertaccessbindingjs">trunk/JSTests/modules/module-assert-access-binding.js</a></li>
<li><a href="#trunkJSTestsmodulesmoduleassertaccessnamespacejs">trunk/JSTests/modules/module-assert-access-namespace.js</a></li>
<li><a href="#trunkJSTestsmodulesnamespaceemptyjs">trunk/JSTests/modules/namespace-empty.js</a></li>
<li><a href="#trunkJSTestsstressmodulenamespaceaccesschangejs">trunk/JSTests/stress/module-namespace-access-change.js</a></li>
<li><a href="#trunkJSTestsstressmodulenamespaceaccessnonconstantjs">trunk/JSTests/stress/module-namespace-access-non-constant.js</a></li>
<li><a href="#trunkJSTestsstressmodulenamespaceaccesspolyjs">trunk/JSTests/stress/module-namespace-access-poly.js</a></li>
<li><a href="#trunkJSTestsstressmodulenamespaceaccesstransitiveexportsjs">trunk/JSTests/stress/module-namespace-access-transitive-exports.js</a></li>
<li><a href="#trunkJSTestsstressmodulenamespaceaccessjs">trunk/JSTests/stress/module-namespace-access.js</a></li>
<li><a href="#trunkJSTestsstressresourcesmodulenamespaceaccesstransitiveexports2js">trunk/JSTests/stress/resources/module-namespace-access-transitive-exports-2.js</a></li>
<li><a href="#trunkJSTestsstressresourcesmodulenamespaceaccesstransitiveexportsjs">trunk/JSTests/stress/resources/module-namespace-access-transitive-exports.js</a></li>
<li><a href="#trunkJSTestsstressresourcesmodulenamespaceaccessjs">trunk/JSTests/stress/resources/module-namespace-access.js</a></li>
<li><a href="#trunkSourceJavaScriptCorebytecodeModuleNamespaceAccessCasecpp">trunk/Source/JavaScriptCore/bytecode/ModuleNamespaceAccessCase.cpp</a></li>
<li><a href="#trunkSourceJavaScriptCorebytecodeModuleNamespaceAccessCaseh">trunk/Source/JavaScriptCore/bytecode/ModuleNamespaceAccessCase.h</a></li>
</ul>
</div>
<div id="patch">
<h3>Diff</h3>
<a id="trunkJSTestsChangeLog"></a>
<div class="modfile"><h4>Modified: trunk/JSTests/ChangeLog (212716 => 212717)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/JSTests/ChangeLog 2017-02-21 17:25:02 UTC (rev 212716)
+++ trunk/JSTests/ChangeLog 2017-02-21 18:00:43 UTC (rev 212717)
</span><span class="lines">@@ -1,3 +1,15 @@
</span><ins>+2017-02-21 Ryan Haddad <ryanhaddad@apple.com>
+
+ Unreviewed, rolling out r212712.
+
+ This change broke the CLoop build.
+
+ Reverted changeset:
+
+ "JSModuleNamespace object should have IC"
+ https://bugs.webkit.org/show_bug.cgi?id=160590
+ http://trac.webkit.org/changeset/212712
+
</ins><span class="cx"> 2017-02-21 Yusuke Suzuki <utatane.tea@gmail.com>
</span><span class="cx">
</span><span class="cx"> JSModuleNamespace object should have IC
</span></span></pre></div>
<a id="trunkJSTestsmodulesmoduleassertaccessbindingjs"></a>
<div class="delfile"><h4>Deleted: trunk/JSTests/modules/module-assert-access-binding.js (212716 => 212717)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/JSTests/modules/module-assert-access-binding.js 2017-02-21 17:25:02 UTC (rev 212716)
+++ trunk/JSTests/modules/module-assert-access-binding.js 2017-02-21 18:00:43 UTC (rev 212717)
</span><span class="lines">@@ -1,10 +0,0 @@
</span><del>-import {shouldBe} from "./resources/assert.js";
-
-let array = [];
-for (let i = 0; i < 4000000; i++) {
- array.push(i);
-}
-
-for (let i = 0; i < 4000000; i++) {
- shouldBe(array[i], i);
-}
</del></span></pre></div>
<a id="trunkJSTestsmodulesmoduleassertaccessnamespacejs"></a>
<div class="delfile"><h4>Deleted: trunk/JSTests/modules/module-assert-access-namespace.js (212716 => 212717)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/JSTests/modules/module-assert-access-namespace.js 2017-02-21 17:25:02 UTC (rev 212716)
+++ trunk/JSTests/modules/module-assert-access-namespace.js 2017-02-21 18:00:43 UTC (rev 212717)
</span><span class="lines">@@ -1,10 +0,0 @@
</span><del>-import * as assert from "./resources/assert.js";
-
-let array = [];
-for (let i = 0; i < 4000000; i++) {
- array.push(i);
-}
-
-for (let i = 0; i < 4000000; i++) {
- assert.shouldBe(array[i], i);
-}
</del></span></pre></div>
<a id="trunkJSTestsmodulesnamespaceemptyjs"></a>
<div class="delfile"><h4>Deleted: trunk/JSTests/modules/namespace-empty.js (212716 => 212717)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/JSTests/modules/namespace-empty.js 2017-02-21 17:25:02 UTC (rev 212716)
+++ trunk/JSTests/modules/namespace-empty.js 2017-02-21 18:00:43 UTC (rev 212717)
</span><span class="lines">@@ -1,17 +0,0 @@
</span><del>-import * as ns from "./namespace-empty.js"
-import {shouldThrow} from "./resources/assert.js"
-
-function access(ns)
-{
- return ns.test;
-}
-noInline(access);
-
-for (var i = 0; i < 1e3; ++i) {
- shouldThrow(() => {
- access(ns);
- }, `ReferenceError: Cannot access uninitialized variable.`);
-}
-
-
-export let test = 42;
</del></span></pre></div>
<a id="trunkJSTestsstressmodulenamespaceaccesschangejs"></a>
<div class="delfile"><h4>Deleted: trunk/JSTests/stress/module-namespace-access-change.js (212716 => 212717)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/JSTests/stress/module-namespace-access-change.js 2017-02-21 17:25:02 UTC (rev 212716)
+++ trunk/JSTests/stress/module-namespace-access-change.js 2017-02-21 18:00:43 UTC (rev 212717)
</span><span class="lines">@@ -1,19 +0,0 @@
</span><del>-function shouldBe(actual, expected) {
- if (actual !== expected)
- throw new Error('bad value: ' + actual);
-}
-
-function access(namespace)
-{
- return namespace.test;
-}
-noInline(access);
-
-import("./resources/module-namespace-access.js").then((ns) => {
- for (var i = 0; i < 1e4; ++i)
- shouldBe(access(ns), 42)
- ns.change();
- for (var i = 0; i < 1e4; ++i)
- shouldBe(access(ns), 55)
-});
-drainMicrotasks();
</del></span></pre></div>
<a id="trunkJSTestsstressmodulenamespaceaccessnonconstantjs"></a>
<div class="delfile"><h4>Deleted: trunk/JSTests/stress/module-namespace-access-non-constant.js (212716 => 212717)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/JSTests/stress/module-namespace-access-non-constant.js 2017-02-21 17:25:02 UTC (rev 212716)
+++ trunk/JSTests/stress/module-namespace-access-non-constant.js 2017-02-21 18:00:43 UTC (rev 212717)
</span><span class="lines">@@ -1,13 +0,0 @@
</span><del>-function shouldBe(actual, expected) {
- if (actual !== expected)
- throw new Error('bad value: ' + actual);
-}
-
-import("./resources/module-namespace-access.js").then((ns) => {
- ns.change();
- for (var i = 0; i < 1e6; ++i) {
- shouldBe(ns.test, 55);
- shouldBe(ns.cocoa(), 55);
- }
-});
-drainMicrotasks();
</del></span></pre></div>
<a id="trunkJSTestsstressmodulenamespaceaccesspolyjs"></a>
<div class="delfile"><h4>Deleted: trunk/JSTests/stress/module-namespace-access-poly.js (212716 => 212717)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/JSTests/stress/module-namespace-access-poly.js 2017-02-21 17:25:02 UTC (rev 212716)
+++ trunk/JSTests/stress/module-namespace-access-poly.js 2017-02-21 18:00:43 UTC (rev 212717)
</span><span class="lines">@@ -1,24 +0,0 @@
</span><del>-function shouldBe(actual, expected) {
- if (actual !== expected)
- throw new Error('bad value: ' + actual);
-}
-
-function access(ns)
-{
- return ns.test;
-}
-noInline(access);
-
-import("./resources/module-namespace-access.js").then((ns) => {
- for (var i = 0; i < 1e4; ++i) {
- shouldBe(access(ns), 42);
- }
- let nonNS = { test: 50 };
- let nonNS2 = { ok: 22, test: 52 };
- for (var i = 0; i < 1e4; ++i) {
- shouldBe(access(ns), 42);
- shouldBe(access(nonNS), 50);
- shouldBe(access(nonNS2), 52);
- }
-});
-drainMicrotasks();
</del></span></pre></div>
<a id="trunkJSTestsstressmodulenamespaceaccesstransitiveexportsjs"></a>
<div class="delfile"><h4>Deleted: trunk/JSTests/stress/module-namespace-access-transitive-exports.js (212716 => 212717)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/JSTests/stress/module-namespace-access-transitive-exports.js 2017-02-21 17:25:02 UTC (rev 212716)
+++ trunk/JSTests/stress/module-namespace-access-transitive-exports.js 2017-02-21 18:00:43 UTC (rev 212717)
</span><span class="lines">@@ -1,12 +0,0 @@
</span><del>-function shouldBe(actual, expected) {
- if (actual !== expected)
- throw new Error('bad value: ' + actual);
-}
-
-import("./resources/module-namespace-access-transitive-exports.js").then((ns) => {
- for (var i = 0; i < 1e6; ++i) {
- shouldBe(ns.test, 42);
- shouldBe(ns.cocoa(), 42);
- }
-});
-drainMicrotasks();
</del></span></pre></div>
<a id="trunkJSTestsstressmodulenamespaceaccessjs"></a>
<div class="delfile"><h4>Deleted: trunk/JSTests/stress/module-namespace-access.js (212716 => 212717)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/JSTests/stress/module-namespace-access.js 2017-02-21 17:25:02 UTC (rev 212716)
+++ trunk/JSTests/stress/module-namespace-access.js 2017-02-21 18:00:43 UTC (rev 212717)
</span><span class="lines">@@ -1,12 +0,0 @@
</span><del>-function shouldBe(actual, expected) {
- if (actual !== expected)
- throw new Error('bad value: ' + actual);
-}
-
-import("./resources/module-namespace-access.js").then((ns) => {
- for (var i = 0; i < 1e6; ++i) {
- shouldBe(ns.test, 42);
- shouldBe(ns.cocoa(), 42);
- }
-});
-drainMicrotasks();
</del></span></pre></div>
<a id="trunkJSTestsstressresourcesmodulenamespaceaccesstransitiveexports2js"></a>
<div class="delfile"><h4>Deleted: trunk/JSTests/stress/resources/module-namespace-access-transitive-exports-2.js (212716 => 212717)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/JSTests/stress/resources/module-namespace-access-transitive-exports-2.js 2017-02-21 17:25:02 UTC (rev 212716)
+++ trunk/JSTests/stress/resources/module-namespace-access-transitive-exports-2.js 2017-02-21 18:00:43 UTC (rev 212717)
</span><span class="lines">@@ -1,10 +0,0 @@
</span><del>-export let test = 42;
-export function cocoa()
-{
- return test;
-}
-
-export function change()
-{
- test = 55;
-}
</del></span></pre></div>
<a id="trunkJSTestsstressresourcesmodulenamespaceaccesstransitiveexportsjs"></a>
<div class="delfile"><h4>Deleted: trunk/JSTests/stress/resources/module-namespace-access-transitive-exports.js (212716 => 212717)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/JSTests/stress/resources/module-namespace-access-transitive-exports.js 2017-02-21 17:25:02 UTC (rev 212716)
+++ trunk/JSTests/stress/resources/module-namespace-access-transitive-exports.js 2017-02-21 18:00:43 UTC (rev 212717)
</span><span class="lines">@@ -1 +0,0 @@
</span><del>-export * from "./module-namespace-access-transitive-exports-2.js"
</del></span></pre></div>
<a id="trunkJSTestsstressresourcesmodulenamespaceaccessjs"></a>
<div class="delfile"><h4>Deleted: trunk/JSTests/stress/resources/module-namespace-access.js (212716 => 212717)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/JSTests/stress/resources/module-namespace-access.js 2017-02-21 17:25:02 UTC (rev 212716)
+++ trunk/JSTests/stress/resources/module-namespace-access.js 2017-02-21 18:00:43 UTC (rev 212717)
</span><span class="lines">@@ -1,10 +0,0 @@
</span><del>-export let test = 42;
-export function cocoa()
-{
- return test;
-}
-
-export function change()
-{
- test = 55;
-}
</del></span></pre></div>
<a id="trunkSourceJavaScriptCoreCMakeListstxt"></a>
<div class="modfile"><h4>Modified: trunk/Source/JavaScriptCore/CMakeLists.txt (212716 => 212717)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/JavaScriptCore/CMakeLists.txt 2017-02-21 17:25:02 UTC (rev 212716)
+++ trunk/Source/JavaScriptCore/CMakeLists.txt 2017-02-21 18:00:43 UTC (rev 212717)
</span><span class="lines">@@ -228,7 +228,6 @@
</span><span class="cx"> bytecode/LLIntPrototypeLoadAdaptiveStructureWatchpoint.cpp
</span><span class="cx"> bytecode/LazyOperandValueProfile.cpp
</span><span class="cx"> bytecode/MethodOfGettingAValueProfile.cpp
</span><del>- bytecode/ModuleNamespaceAccessCase.cpp
</del><span class="cx"> bytecode/ModuleProgramCodeBlock.cpp
</span><span class="cx"> bytecode/ObjectPropertyCondition.cpp
</span><span class="cx"> bytecode/ObjectPropertyConditionSet.cpp
</span></span></pre></div>
<a id="trunkSourceJavaScriptCoreChangeLog"></a>
<div class="modfile"><h4>Modified: trunk/Source/JavaScriptCore/ChangeLog (212716 => 212717)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/JavaScriptCore/ChangeLog 2017-02-21 17:25:02 UTC (rev 212716)
+++ trunk/Source/JavaScriptCore/ChangeLog 2017-02-21 18:00:43 UTC (rev 212717)
</span><span class="lines">@@ -1,3 +1,15 @@
</span><ins>+2017-02-21 Ryan Haddad <ryanhaddad@apple.com>
+
+ Unreviewed, rolling out r212712.
+
+ This change broke the CLoop build.
+
+ Reverted changeset:
+
+ "JSModuleNamespace object should have IC"
+ https://bugs.webkit.org/show_bug.cgi?id=160590
+ http://trac.webkit.org/changeset/212712
+
</ins><span class="cx"> 2017-02-21 Yusuke Suzuki <utatane.tea@gmail.com>
</span><span class="cx">
</span><span class="cx"> JSModuleNamespace object should have IC
</span></span></pre></div>
<a id="trunkSourceJavaScriptCoreJavaScriptCorexcodeprojprojectpbxproj"></a>
<div class="modfile"><h4>Modified: trunk/Source/JavaScriptCore/JavaScriptCore.xcodeproj/project.pbxproj (212716 => 212717)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/JavaScriptCore/JavaScriptCore.xcodeproj/project.pbxproj 2017-02-21 17:25:02 UTC (rev 212716)
+++ trunk/Source/JavaScriptCore/JavaScriptCore.xcodeproj/project.pbxproj 2017-02-21 18:00:43 UTC (rev 212717)
</span><span class="lines">@@ -2405,8 +2405,6 @@
</span><span class="cx"> FED94F2F171E3E2300BE77A4 /* Watchdog.h in Headers */ = {isa = PBXBuildFile; fileRef = FED94F2C171E3E2300BE77A4 /* Watchdog.h */; settings = {ATTRIBUTES = (Private, ); }; };
</span><span class="cx"> FEF040511AAE662D00BD28B0 /* CompareAndSwapTest.cpp in Sources */ = {isa = PBXBuildFile; fileRef = FEF040501AAE662D00BD28B0 /* CompareAndSwapTest.cpp */; };
</span><span class="cx"> FEFD6FC61D5E7992008F2F0B /* JSStringInlines.h in Headers */ = {isa = PBXBuildFile; fileRef = FEFD6FC51D5E7970008F2F0B /* JSStringInlines.h */; settings = {ATTRIBUTES = (Private, ); }; };
</span><del>- 321D9E4CFB67423A97F191A7 /* ModuleNamespaceAccessCase.cpp in Sources */ = {isa = PBXBuildFile; fileRef = 20ECB15EFC524624BC2F02D5 /* ModuleNamespaceAccessCase.cpp */; };
- 9F63434577274FAFB9336C38 /* ModuleNamespaceAccessCase.h in Headers */ = {isa = PBXBuildFile; fileRef = 4CE978E385A8498199052153 /* ModuleNamespaceAccessCase.h */; };
</del><span class="cx"> /* End PBXBuildFile section */
</span><span class="cx">
</span><span class="cx"> /* Begin PBXContainerItemProxy section */
</span><span class="lines">@@ -4968,8 +4966,6 @@
</span><span class="cx"> FEF040501AAE662D00BD28B0 /* CompareAndSwapTest.cpp */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp; name = CompareAndSwapTest.cpp; path = API/tests/CompareAndSwapTest.cpp; sourceTree = "<group>"; };
</span><span class="cx"> FEF040521AAEC4ED00BD28B0 /* CompareAndSwapTest.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; name = CompareAndSwapTest.h; path = API/tests/CompareAndSwapTest.h; sourceTree = "<group>"; };
</span><span class="cx"> FEFD6FC51D5E7970008F2F0B /* JSStringInlines.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = JSStringInlines.h; sourceTree = "<group>"; };
</span><del>- 20ECB15EFC524624BC2F02D5 /* ModuleNamespaceAccessCase.cpp */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp; name = ModuleNamespaceAccessCase.cpp; path = ModuleNamespaceAccessCase.cpp; sourceTree = "<group>"; };
- 4CE978E385A8498199052153 /* ModuleNamespaceAccessCase.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; name = ModuleNamespaceAccessCase.h; path = ModuleNamespaceAccessCase.h; sourceTree = "<group>"; };
</del><span class="cx"> /* End PBXFileReference section */
</span><span class="cx">
</span><span class="cx"> /* Begin PBXFrameworksBuildPhase section */
</span><span class="lines">@@ -7503,8 +7499,6 @@
</span><span class="cx"> 0F426A461460CBAB00131F8F /* VirtualRegister.h */,
</span><span class="cx"> 0F919D2215853CDE004A4E7D /* Watchpoint.cpp */,
</span><span class="cx"> 0F919D2315853CDE004A4E7D /* Watchpoint.h */,
</span><del>- 20ECB15EFC524624BC2F02D5 /* ModuleNamespaceAccessCase.cpp */,
- 4CE978E385A8498199052153 /* ModuleNamespaceAccessCase.h */,
</del><span class="cx"> );
</span><span class="cx"> path = bytecode;
</span><span class="cx"> sourceTree = "<group>";
</span><span class="lines">@@ -9254,7 +9248,6 @@
</span><span class="cx"> 86704B8812DBA33700A9FE7B /* YarrParser.h in Headers */,
</span><span class="cx"> 86704B8A12DBA33700A9FE7B /* YarrPattern.h in Headers */,
</span><span class="cx"> 86704B4312DB8A8100A9FE7B /* YarrSyntaxChecker.h in Headers */,
</span><del>- 9F63434577274FAFB9336C38 /* ModuleNamespaceAccessCase.h in Headers */,
</del><span class="cx"> );
</span><span class="cx"> runOnlyForDeploymentPostprocessing = 0;
</span><span class="cx"> };
</span><span class="lines">@@ -10657,7 +10650,6 @@
</span><span class="cx"> 86704B8612DBA33700A9FE7B /* YarrJIT.cpp in Sources */,
</span><span class="cx"> 86704B8912DBA33700A9FE7B /* YarrPattern.cpp in Sources */,
</span><span class="cx"> 86704B4212DB8A8100A9FE7B /* YarrSyntaxChecker.cpp in Sources */,
</span><del>- 321D9E4CFB67423A97F191A7 /* ModuleNamespaceAccessCase.cpp in Sources */,
</del><span class="cx"> );
</span><span class="cx"> runOnlyForDeploymentPostprocessing = 0;
</span><span class="cx"> };
</span></span></pre></div>
<a id="trunkSourceJavaScriptCorebytecodeAccessCasecpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/JavaScriptCore/bytecode/AccessCase.cpp (212716 => 212717)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/JavaScriptCore/bytecode/AccessCase.cpp 2017-02-21 17:25:02 UTC (rev 212716)
+++ trunk/Source/JavaScriptCore/bytecode/AccessCase.cpp 2017-02-21 18:00:43 UTC (rev 212717)
</span><span class="lines">@@ -37,10 +37,7 @@
</span><span class="cx"> #include "HeapInlines.h"
</span><span class="cx"> #include "IntrinsicGetterAccessCase.h"
</span><span class="cx"> #include "JSCJSValueInlines.h"
</span><del>-#include "JSModuleEnvironment.h"
-#include "JSModuleNamespaceObject.h"
</del><span class="cx"> #include "LinkBuffer.h"
</span><del>-#include "ModuleNamespaceAccessCase.h"
</del><span class="cx"> #include "PolymorphicAccess.h"
</span><span class="cx"> #include "ScopedArguments.h"
</span><span class="cx"> #include "ScratchRegisterAllocator.h"
</span><span class="lines">@@ -68,7 +65,6 @@
</span><span class="cx"> case StringLength:
</span><span class="cx"> case DirectArgumentsLength:
</span><span class="cx"> case ScopedArgumentsLength:
</span><del>- case ModuleNamespaceLoad:
</del><span class="cx"> case Replace:
</span><span class="cx"> break;
</span><span class="cx"> default:
</span><span class="lines">@@ -152,7 +148,6 @@
</span><span class="cx"> case StringLength:
</span><span class="cx"> case DirectArgumentsLength:
</span><span class="cx"> case ScopedArgumentsLength:
</span><del>- case ModuleNamespaceLoad:
</del><span class="cx"> return false;
</span><span class="cx"> default:
</span><span class="cx"> return true;
</span><span class="lines">@@ -198,13 +193,6 @@
</span><span class="cx"> case DirectArgumentsLength:
</span><span class="cx"> case ScopedArgumentsLength:
</span><span class="cx"> return other.type() == type();
</span><del>- case ModuleNamespaceLoad: {
- if (other.type() != type())
- return false;
- auto& thisCase = this->as<ModuleNamespaceAccessCase>();
- auto& otherCase = this->as<ModuleNamespaceAccessCase>();
- return thisCase.moduleNamespaceObject() == otherCase.moduleNamespaceObject();
- }
</del><span class="cx"> default:
</span><span class="cx"> if (!guardedByStructureCheck() || !other.guardedByStructureCheck())
</span><span class="cx"> return false;
</span><span class="lines">@@ -251,12 +239,6 @@
</span><span class="cx"> auto& intrinsic = this->as<IntrinsicGetterAccessCase>();
</span><span class="cx"> if (intrinsic.intrinsicFunction() && !Heap::isMarked(intrinsic.intrinsicFunction()))
</span><span class="cx"> return false;
</span><del>- } else if (type() == ModuleNamespaceLoad) {
- auto& accessCase = this->as<ModuleNamespaceAccessCase>();
- if (accessCase.moduleNamespaceObject() && !Heap::isMarked(accessCase.moduleNamespaceObject()))
- return false;
- if (accessCase.moduleEnvironment() && !Heap::isMarked(accessCase.moduleEnvironment()))
- return false;
</del><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> return true;
</span><span class="lines">@@ -362,11 +344,6 @@
</span><span class="cx"> return;
</span><span class="cx"> }
</span><span class="cx">
</span><del>- case ModuleNamespaceLoad: {
- this->as<ModuleNamespaceAccessCase>().emit(state, fallThrough);
- return;
- }
-
</del><span class="cx"> default: {
</span><span class="cx"> if (viaProxy()) {
</span><span class="cx"> fallThrough.append(
</span><span class="lines">@@ -1014,7 +991,6 @@
</span><span class="cx">
</span><span class="cx"> case DirectArgumentsLength:
</span><span class="cx"> case ScopedArgumentsLength:
</span><del>- case ModuleNamespaceLoad:
</del><span class="cx"> // These need to be handled by generateWithGuard(), since the guard is part of the
</span><span class="cx"> // algorithm. We can be sure that nobody will call generate() directly for these since they
</span><span class="cx"> // are not guarded by structure checks.
</span></span></pre></div>
<a id="trunkSourceJavaScriptCorebytecodeAccessCaseh"></a>
<div class="modfile"><h4>Modified: trunk/Source/JavaScriptCore/bytecode/AccessCase.h (212716 => 212717)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/JavaScriptCore/bytecode/AccessCase.h 2017-02-21 17:25:02 UTC (rev 212716)
+++ trunk/Source/JavaScriptCore/bytecode/AccessCase.h 2017-02-21 18:00:43 UTC (rev 212717)
</span><span class="lines">@@ -95,8 +95,7 @@
</span><span class="cx"> ArrayLength,
</span><span class="cx"> StringLength,
</span><span class="cx"> DirectArgumentsLength,
</span><del>- ScopedArgumentsLength,
- ModuleNamespaceLoad,
</del><ins>+ ScopedArgumentsLength
</ins><span class="cx"> };
</span><span class="cx">
</span><span class="cx"> enum State : uint8_t {
</span></span></pre></div>
<a id="trunkSourceJavaScriptCorebytecodeGetByIdStatuscpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/JavaScriptCore/bytecode/GetByIdStatus.cpp (212716 => 212717)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/JavaScriptCore/bytecode/GetByIdStatus.cpp 2017-02-21 17:25:02 UTC (rev 212716)
+++ trunk/Source/JavaScriptCore/bytecode/GetByIdStatus.cpp 2017-02-21 18:00:43 UTC (rev 212717)
</span><span class="lines">@@ -34,7 +34,6 @@
</span><span class="cx"> #include "JSScope.h"
</span><span class="cx"> #include "LLIntData.h"
</span><span class="cx"> #include "LowLevelInterpreter.h"
</span><del>-#include "ModuleNamespaceAccessCase.h"
</del><span class="cx"> #include "PolymorphicAccess.h"
</span><span class="cx"> #include "StructureStubInfo.h"
</span><span class="cx"> #include <wtf/ListDump.h>
</span><span class="lines">@@ -44,15 +43,6 @@
</span><span class="cx"> class GetterSetter;
</span><span class="cx"> }
</span><span class="cx">
</span><del>-GetByIdStatus::GetByIdStatus(const ModuleNamespaceAccessCase& accessCase)
- : m_state(ModuleNamespace)
- , m_wasSeenInJIT(true)
- , m_moduleNamespaceObject(accessCase.moduleNamespaceObject())
- , m_moduleEnvironment(accessCase.moduleEnvironment())
- , m_scopeOffset(accessCase.scopeOffset())
-{
-}
-
</del><span class="cx"> bool GetByIdStatus::appendVariant(const GetByIdVariant& variant)
</span><span class="cx"> {
</span><span class="cx"> // Attempt to merge this variant with an already existing variant.
</span><span class="lines">@@ -205,16 +195,6 @@
</span><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> case CacheType::Stub: {
</span><del>- if (list->size() == 1) {
- const AccessCase& access = list->at(0);
- switch (access.type()) {
- case AccessCase::ModuleNamespaceLoad:
- return GetByIdStatus(access.as<ModuleNamespaceAccessCase>());
- default:
- break;
- }
- }
-
</del><span class="cx"> for (unsigned listIndex = 0; listIndex < list->size(); ++listIndex) {
</span><span class="cx"> const AccessCase& access = list->at(listIndex);
</span><span class="cx"> if (access.viaProxy())
</span><span class="lines">@@ -396,7 +376,6 @@
</span><span class="cx"> case NoInformation:
</span><span class="cx"> case TakesSlowPath:
</span><span class="cx"> case Custom:
</span><del>- case ModuleNamespace:
</del><span class="cx"> return false;
</span><span class="cx"> case Simple:
</span><span class="cx"> for (unsigned i = m_variants.size(); i--;) {
</span><span class="lines">@@ -441,9 +420,6 @@
</span><span class="cx"> case Custom:
</span><span class="cx"> out.print("Custom");
</span><span class="cx"> break;
</span><del>- case ModuleNamespace:
- out.print("ModuleNamespace");
- break;
</del><span class="cx"> case TakesSlowPath:
</span><span class="cx"> out.print("TakesSlowPath");
</span><span class="cx"> break;
</span></span></pre></div>
<a id="trunkSourceJavaScriptCorebytecodeGetByIdStatush"></a>
<div class="modfile"><h4>Modified: trunk/Source/JavaScriptCore/bytecode/GetByIdStatus.h (212716 => 212717)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/JavaScriptCore/bytecode/GetByIdStatus.h 2017-02-21 17:25:02 UTC (rev 212716)
+++ trunk/Source/JavaScriptCore/bytecode/GetByIdStatus.h 2017-02-21 18:00:43 UTC (rev 212717)
</span><span class="lines">@@ -30,15 +30,10 @@
</span><span class="cx"> #include "ConcurrentJSLock.h"
</span><span class="cx"> #include "ExitingJITType.h"
</span><span class="cx"> #include "GetByIdVariant.h"
</span><del>-#include "ScopeOffset.h"
</del><span class="cx">
</span><span class="cx"> namespace JSC {
</span><span class="cx">
</span><del>-class AccessCase;
</del><span class="cx"> class CodeBlock;
</span><del>-class JSModuleEnvironment;
-class JSModuleNamespaceObject;
-class ModuleNamespaceAccessCase;
</del><span class="cx"> class StructureStubInfo;
</span><span class="cx">
</span><span class="cx"> typedef HashMap<CodeOrigin, StructureStubInfo*, CodeOriginApproximateHash> StubInfoMap;
</span><span class="lines">@@ -46,19 +41,12 @@
</span><span class="cx"> class GetByIdStatus {
</span><span class="cx"> public:
</span><span class="cx"> enum State {
</span><del>- // It's uncached so we have no information.
- NoInformation,
- // It's cached for a simple access to a known object property with
- // a possible structure chain and a possible specific value.
- Simple,
- // It's cached for a custom accessor with a possible structure chain.
- Custom,
- // It's cached for an access to a module namespace object's binding.
- ModuleNamespace,
- // It's known to often take slow path.
- TakesSlowPath,
- // It's known to take paths that make calls.
- MakesCalls,
</del><ins>+ NoInformation, // It's uncached so we have no information.
+ Simple, // It's cached for a simple access to a known object property with
+ // a possible structure chain and a possible specific value.
+ Custom, // It's cached for a custom accessor with a possible structure chain.
+ TakesSlowPath, // It's known to often take slow path.
+ MakesCalls // It's known to take paths that make calls.
</ins><span class="cx"> };
</span><span class="cx">
</span><span class="cx"> GetByIdStatus()
</span><span class="lines">@@ -71,8 +59,6 @@
</span><span class="cx"> {
</span><span class="cx"> ASSERT(state == NoInformation || state == TakesSlowPath || state == MakesCalls);
</span><span class="cx"> }
</span><del>-
- GetByIdStatus(const ModuleNamespaceAccessCase&);
</del><span class="cx">
</span><span class="cx"> GetByIdStatus(
</span><span class="cx"> State state, bool wasSeenInJIT, const GetByIdVariant& variant = GetByIdVariant())
</span><span class="lines">@@ -98,7 +84,6 @@
</span><span class="cx"> bool operator!() const { return !isSet(); }
</span><span class="cx"> bool isSimple() const { return m_state == Simple; }
</span><span class="cx"> bool isCustom() const { return m_state == Custom; }
</span><del>- bool isModuleNamespace() const { return m_state == ModuleNamespace; }
</del><span class="cx">
</span><span class="cx"> size_t numVariants() const { return m_variants.size(); }
</span><span class="cx"> const Vector<GetByIdVariant, 1>& variants() const { return m_variants; }
</span><span class="lines">@@ -105,7 +90,7 @@
</span><span class="cx"> const GetByIdVariant& at(size_t index) const { return m_variants[index]; }
</span><span class="cx"> const GetByIdVariant& operator[](size_t index) const { return at(index); }
</span><span class="cx">
</span><del>- bool takesSlowPath() const { return m_state == TakesSlowPath || m_state == MakesCalls || m_state == Custom || m_state == ModuleNamespace; }
</del><ins>+ bool takesSlowPath() const { return m_state == TakesSlowPath || m_state == MakesCalls || m_state == Custom; }
</ins><span class="cx"> bool makesCalls() const;
</span><span class="cx">
</span><span class="cx"> bool wasSeenInJIT() const { return m_wasSeenInJIT; }
</span><span class="lines">@@ -112,10 +97,6 @@
</span><span class="cx">
</span><span class="cx"> // Attempts to reduce the set of variants to fit the given structure set. This may be approximate.
</span><span class="cx"> void filter(const StructureSet&);
</span><del>-
- JSModuleNamespaceObject* moduleNamespaceObject() const { return m_moduleNamespaceObject; }
- JSModuleEnvironment* moduleEnvironment() const { return m_moduleEnvironment; }
- ScopeOffset scopeOffset() const { return m_scopeOffset; }
</del><span class="cx">
</span><span class="cx"> void dump(PrintStream&) const;
</span><span class="cx">
</span><span class="lines">@@ -135,9 +116,6 @@
</span><span class="cx"> State m_state;
</span><span class="cx"> Vector<GetByIdVariant, 1> m_variants;
</span><span class="cx"> bool m_wasSeenInJIT;
</span><del>- JSModuleNamespaceObject* m_moduleNamespaceObject { nullptr };
- JSModuleEnvironment* m_moduleEnvironment { nullptr };
- ScopeOffset m_scopeOffset { };
</del><span class="cx"> };
</span><span class="cx">
</span><span class="cx"> } // namespace JSC
</span></span></pre></div>
<a id="trunkSourceJavaScriptCorebytecodeModuleNamespaceAccessCasecpp"></a>
<div class="delfile"><h4>Deleted: trunk/Source/JavaScriptCore/bytecode/ModuleNamespaceAccessCase.cpp (212716 => 212717)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/JavaScriptCore/bytecode/ModuleNamespaceAccessCase.cpp 2017-02-21 17:25:02 UTC (rev 212716)
+++ trunk/Source/JavaScriptCore/bytecode/ModuleNamespaceAccessCase.cpp 2017-02-21 18:00:43 UTC (rev 212717)
</span><span class="lines">@@ -1,84 +0,0 @@
</span><del>-/*
- * Copyright (C) 2017 Yusuke Suzuki <utatane.tea@gmail.com>.
- *
- * 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 "ModuleNamespaceAccessCase.h"
-
-#if ENABLE(JIT)
-
-#include "CCallHelpers.h"
-#include "HeapInlines.h"
-#include "JSModuleEnvironment.h"
-#include "JSModuleNamespaceObject.h"
-#include "PolymorphicAccess.h"
-#include "StructureStubInfo.h"
-
-namespace JSC {
-
-ModuleNamespaceAccessCase::ModuleNamespaceAccessCase(VM& vm, JSCell* owner, JSModuleNamespaceObject* moduleNamespaceObject, JSModuleEnvironment* moduleEnvironment, ScopeOffset scopeOffset)
- : Base(vm, owner, ModuleNamespaceLoad, invalidOffset, nullptr, ObjectPropertyConditionSet())
- , m_scopeOffset(scopeOffset)
-{
- m_moduleNamespaceObject.set(vm, owner, moduleNamespaceObject);
- m_moduleEnvironment.set(vm, owner, moduleEnvironment);
-}
-
-std::unique_ptr<AccessCase> ModuleNamespaceAccessCase::create(VM& vm, JSCell* owner, JSModuleNamespaceObject* moduleNamespaceObject, JSModuleEnvironment* moduleEnvironment, ScopeOffset scopeOffset)
-{
- return std::unique_ptr<AccessCase>(new ModuleNamespaceAccessCase(vm, owner, moduleNamespaceObject, moduleEnvironment, scopeOffset));
-}
-
-ModuleNamespaceAccessCase::~ModuleNamespaceAccessCase()
-{
-}
-
-std::unique_ptr<AccessCase> ModuleNamespaceAccessCase::clone() const
-{
- std::unique_ptr<ModuleNamespaceAccessCase> result(new ModuleNamespaceAccessCase(*this));
- result->resetState();
- return WTFMove(result);
-}
-
-void ModuleNamespaceAccessCase::emit(AccessGenerationState& state, MacroAssembler::JumpList& fallThrough)
-{
- CCallHelpers& jit = *state.jit;
- JSValueRegs valueRegs = state.valueRegs;
- GPRReg baseGPR = state.baseGPR;
-
- fallThrough.append(
- jit.branchPtr(
- CCallHelpers::NotEqual,
- baseGPR,
- CCallHelpers::TrustedImmPtr(m_moduleNamespaceObject.get())));
-
- jit.loadValue(&m_moduleEnvironment->variableAt(m_scopeOffset), valueRegs);
- state.failAndIgnore.append(jit.branchIfEmpty(valueRegs));
- state.succeed();
-}
-
-
-} // namespace JSC
-
-#endif // ENABLE(JIT)
</del></span></pre></div>
<a id="trunkSourceJavaScriptCorebytecodeModuleNamespaceAccessCaseh"></a>
<div class="delfile"><h4>Deleted: trunk/Source/JavaScriptCore/bytecode/ModuleNamespaceAccessCase.h (212716 => 212717)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/JavaScriptCore/bytecode/ModuleNamespaceAccessCase.h 2017-02-21 17:25:02 UTC (rev 212716)
+++ trunk/Source/JavaScriptCore/bytecode/ModuleNamespaceAccessCase.h 2017-02-21 18:00:43 UTC (rev 212717)
</span><span class="lines">@@ -1,64 +0,0 @@
</span><del>-/*
- * Copyright (C) 2017 Yusuke Suzuki <utatane.tea@gmail.com>.
- *
- * 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.
- */
-
-#pragma once
-
-#if ENABLE(JIT)
-
-#include "AccessCase.h"
-
-namespace JSC {
-
-class JSModuleEnvironment;
-class JSModuleNamespaceObject;
-
-class ModuleNamespaceAccessCase : public AccessCase {
-public:
- using Base = AccessCase;
- friend class AccessCase;
-
- JSModuleNamespaceObject* moduleNamespaceObject() const { return m_moduleNamespaceObject.get(); }
- JSModuleEnvironment* moduleEnvironment() const { return m_moduleEnvironment.get(); }
- ScopeOffset scopeOffset() const { return m_scopeOffset; }
-
- static std::unique_ptr<AccessCase> create(VM&, JSCell* owner, JSModuleNamespaceObject*, JSModuleEnvironment*, ScopeOffset);
-
- std::unique_ptr<AccessCase> clone() const override;
-
- void emit(AccessGenerationState&, MacroAssembler::JumpList& fallThrough);
-
- ~ModuleNamespaceAccessCase();
-
-private:
- ModuleNamespaceAccessCase(VM&, JSCell* owner, JSModuleNamespaceObject*, JSModuleEnvironment*, ScopeOffset);
-
- WriteBarrier<JSModuleNamespaceObject> m_moduleNamespaceObject;
- WriteBarrier<JSModuleEnvironment> m_moduleEnvironment;
- ScopeOffset m_scopeOffset;
-};
-
-} // namespace JSC
-
-#endif // ENABLE(JIT)
</del></span></pre></div>
<a id="trunkSourceJavaScriptCorebytecodePolymorphicAccesscpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/JavaScriptCore/bytecode/PolymorphicAccess.cpp (212716 => 212717)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/JavaScriptCore/bytecode/PolymorphicAccess.cpp 2017-02-21 17:25:02 UTC (rev 212716)
+++ trunk/Source/JavaScriptCore/bytecode/PolymorphicAccess.cpp 2017-02-21 18:00:43 UTC (rev 212717)
</span><span class="lines">@@ -645,9 +645,6 @@
</span><span class="cx"> case AccessCase::ScopedArgumentsLength:
</span><span class="cx"> out.print("ScopedArgumentsLength");
</span><span class="cx"> return;
</span><del>- case AccessCase::ModuleNamespaceLoad:
- out.print("ModuleNamespaceLoad");
- return;
</del><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> RELEASE_ASSERT_NOT_REACHED();
</span></span></pre></div>
<a id="trunkSourceJavaScriptCoredfgDFGByteCodeParsercpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/JavaScriptCore/dfg/DFGByteCodeParser.cpp (212716 => 212717)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/JavaScriptCore/dfg/DFGByteCodeParser.cpp 2017-02-21 17:25:02 UTC (rev 212716)
+++ trunk/Source/JavaScriptCore/dfg/DFGByteCodeParser.cpp 2017-02-21 18:00:43 UTC (rev 212717)
</span><span class="lines">@@ -46,7 +46,6 @@
</span><span class="cx"> #include "Heap.h"
</span><span class="cx"> #include "JSCInlines.h"
</span><span class="cx"> #include "JSModuleEnvironment.h"
</span><del>-#include "JSModuleNamespaceObject.h"
</del><span class="cx"> #include "NumberConstructor.h"
</span><span class="cx"> #include "ObjectConstructor.h"
</span><span class="cx"> #include "PreciseJumpTargets.h"
</span><span class="lines">@@ -224,7 +223,6 @@
</span><span class="cx"> Node* handlePutByOffset(Node* base, unsigned identifier, PropertyOffset, const InferredType::Descriptor&, Node* value);
</span><span class="cx"> Node* handleGetByOffset(SpeculatedType, Node* base, unsigned identifierNumber, PropertyOffset, const InferredType::Descriptor&, NodeType = GetByOffset);
</span><span class="cx"> bool handleDOMJITGetter(int resultOperand, const GetByIdVariant&, Node* thisNode, unsigned identifierNumber, SpeculatedType prediction);
</span><del>- bool handleModuleNamespaceLoad(int resultOperand, SpeculatedType, Node* base, GetByIdStatus);
</del><span class="cx">
</span><span class="cx"> // Create a presence ObjectPropertyCondition based on some known offset and structure set. Does not
</span><span class="cx"> // check the validity of the condition, but it may return a null one if it encounters a contradiction.
</span><span class="lines">@@ -2846,34 +2844,6 @@
</span><span class="cx"> return true;
</span><span class="cx"> }
</span><span class="cx">
</span><del>-bool ByteCodeParser::handleModuleNamespaceLoad(int resultOperand, SpeculatedType prediction, Node* base, GetByIdStatus getById)
-{
- if (m_inlineStackTop->m_exitProfile.hasExitSite(m_currentIndex, BadCell))
- return false;
- addToGraph(CheckCell, OpInfo(m_graph.freeze(getById.moduleNamespaceObject())), Edge(base, CellUse));
-
- // Ideally we wouldn't have to do this Phantom. But:
- //
- // For the constant case: we must do it because otherwise we would have no way of knowing
- // that the scope is live at OSR here.
- //
- // For the non-constant case: GetClosureVar could be DCE'd, but baseline's implementation
- // won't be able to handle an Undefined scope.
- addToGraph(Phantom, base);
-
- // Constant folding in the bytecode parser is important for performance. This may not
- // have executed yet. If it hasn't, then we won't have a prediction. Lacking a
- // prediction, we'd otherwise think that it has to exit. Then when it did execute, we
- // would recompile. But if we can fold it here, we avoid the exit.
- m_graph.freeze(getById.moduleEnvironment());
- if (JSValue value = m_graph.tryGetConstantClosureVar(getById.moduleEnvironment(), getById.scopeOffset())) {
- set(VirtualRegister(resultOperand), weakJSConstant(value));
- return true;
- }
- set(VirtualRegister(resultOperand), addToGraph(GetClosureVar, OpInfo(getById.scopeOffset().offset()), OpInfo(prediction), weakJSConstant(getById.moduleEnvironment())));
- return true;
-}
-
</del><span class="cx"> template<typename ChecksFunctor>
</span><span class="cx"> bool ByteCodeParser::handleTypedArrayConstructor(
</span><span class="cx"> int resultOperand, InternalFunction* function, int registerOffset,
</span><span class="lines">@@ -3468,14 +3438,6 @@
</span><span class="cx"> else
</span><span class="cx"> getById = TryGetById;
</span><span class="cx">
</span><del>- if (getById != TryGetById && getByIdStatus.isModuleNamespace()) {
- if (handleModuleNamespaceLoad(destinationOperand, prediction, base, getByIdStatus)) {
- if (m_graph.compilation())
- m_graph.compilation()->noticeInlinedGetById();
- return;
- }
- }
-
</del><span class="cx"> // Special path for custom accessors since custom's offset does not have any meanings.
</span><span class="cx"> // So, this is completely different from Simple one. But we have a chance to optimize it when we use DOMJIT.
</span><span class="cx"> if (Options::useDOMJIT() && getByIdStatus.isCustom()) {
</span></span></pre></div>
<a id="trunkSourceJavaScriptCorejitAssemblyHelpersh"></a>
<div class="modfile"><h4>Modified: trunk/Source/JavaScriptCore/jit/AssemblyHelpers.h (212716 => 212717)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/JavaScriptCore/jit/AssemblyHelpers.h 2017-02-21 17:25:02 UTC (rev 212716)
+++ trunk/Source/JavaScriptCore/jit/AssemblyHelpers.h 2017-02-21 18:00:43 UTC (rev 212717)
</span><span class="lines">@@ -158,16 +158,6 @@
</span><span class="cx"> }
</span><span class="cx"> #endif
</span><span class="cx"> }
</span><del>-
- void loadValue(void* address, JSValueRegs regs)
- {
-#if USE(JSVALUE64)
- load64(address, regs.gpr());
-#else
- load32(bitwise_cast<void*>(bitwise_cast<uintptr_t>(address) + PayloadOffset), regs.payloadGPR());
- load32(bitwise_cast<void*>(bitwise_cast<uintptr_t>(address) + TagOffset), regs.tagGPR());
-#endif
- }
</del><span class="cx">
</span><span class="cx"> // Note that this clobbers offset.
</span><span class="cx"> void loadProperty(GPRReg object, GPRReg offset, JSValueRegs result);
</span></span></pre></div>
<a id="trunkSourceJavaScriptCorejitRepatchcpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/JavaScriptCore/jit/Repatch.cpp (212716 => 212717)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/JavaScriptCore/jit/Repatch.cpp 2017-02-21 17:25:02 UTC (rev 212716)
+++ trunk/Source/JavaScriptCore/jit/Repatch.cpp 2017-02-21 18:00:43 UTC (rev 212717)
</span><span class="lines">@@ -46,10 +46,8 @@
</span><span class="cx"> #include "JIT.h"
</span><span class="cx"> #include "JITInlines.h"
</span><span class="cx"> #include "JSCInlines.h"
</span><del>-#include "JSModuleNamespaceObject.h"
</del><span class="cx"> #include "JSWebAssembly.h"
</span><span class="cx"> #include "LinkBuffer.h"
</span><del>-#include "ModuleNamespaceAccessCase.h"
</del><span class="cx"> #include "PolymorphicAccess.h"
</span><span class="cx"> #include "ScopedArguments.h"
</span><span class="cx"> #include "ScratchRegisterAllocator.h"
</span><span class="lines">@@ -194,11 +192,6 @@
</span><span class="cx"> newCase = AccessCase::create(vm, codeBlock, AccessCase::ScopedArgumentsLength);
</span><span class="cx"> }
</span><span class="cx"> }
</span><del>-
- if (!propertyName.isSymbol() && isJSModuleNamespaceObject(baseValue) && !slot.isUnset()) {
- if (auto moduleNamespaceSlot = slot.moduleNamespaceSlot())
- newCase = ModuleNamespaceAccessCase::create(vm, codeBlock, jsCast<JSModuleNamespaceObject*>(baseValue), moduleNamespaceSlot->environment, ScopeOffset(moduleNamespaceSlot->scopeOffset));
- }
</del><span class="cx">
</span><span class="cx"> if (!newCase) {
</span><span class="cx"> if (!slot.isCacheable() && !slot.isUnset())
</span></span></pre></div>
<a id="trunkSourceJavaScriptCoreruntimeAbstractModuleRecordcpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/JavaScriptCore/runtime/AbstractModuleRecord.cpp (212716 => 212717)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/JavaScriptCore/runtime/AbstractModuleRecord.cpp 2017-02-21 17:25:02 UTC (rev 212716)
+++ trunk/Source/JavaScriptCore/runtime/AbstractModuleRecord.cpp 2017-02-21 18:00:43 UTC (rev 212717)
</span><span class="lines">@@ -697,10 +697,9 @@
</span><span class="cx"> IdentifierSet exportedNames;
</span><span class="cx"> getExportedNames(exec, this, exportedNames);
</span><span class="cx">
</span><del>- Vector<std::pair<Identifier, Resolution>> resolutions;
</del><ins>+ IdentifierSet unambiguousNames;
</ins><span class="cx"> for (auto& name : exportedNames) {
</span><del>- Identifier ident = Identifier::fromUid(exec, name.get());
- const Resolution resolution = resolveExport(exec, ident);
</del><ins>+ const AbstractModuleRecord::Resolution resolution = resolveExport(exec, Identifier::fromUid(exec, name.get()));
</ins><span class="cx"> switch (resolution.type) {
</span><span class="cx"> case Resolution::Type::NotFound:
</span><span class="cx"> throwSyntaxError(exec, scope, makeString("Exported binding name '", String(name.get()), "' is not found."));
</span><span class="lines">@@ -714,12 +713,12 @@
</span><span class="cx"> break;
</span><span class="cx">
</span><span class="cx"> case Resolution::Type::Resolved:
</span><del>- resolutions.append({ WTFMove(ident), resolution });
</del><ins>+ unambiguousNames.add(name);
</ins><span class="cx"> break;
</span><span class="cx"> }
</span><span class="cx"> }
</span><span class="cx">
</span><del>- m_moduleNamespaceObject.set(vm, this, JSModuleNamespaceObject::create(exec, globalObject, globalObject->moduleNamespaceObjectStructure(), this, WTFMove(resolutions)));
</del><ins>+ m_moduleNamespaceObject.set(vm, this, JSModuleNamespaceObject::create(exec, globalObject, globalObject->moduleNamespaceObjectStructure(), this, unambiguousNames));
</ins><span class="cx"> return m_moduleNamespaceObject.get();
</span><span class="cx"> }
</span><span class="cx">
</span></span></pre></div>
<a id="trunkSourceJavaScriptCoreruntimeJSModuleNamespaceObjectcpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/JavaScriptCore/runtime/JSModuleNamespaceObject.cpp (212716 => 212717)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/JavaScriptCore/runtime/JSModuleNamespaceObject.cpp 2017-02-21 17:25:02 UTC (rev 212716)
+++ trunk/Source/JavaScriptCore/runtime/JSModuleNamespaceObject.cpp 2017-02-21 18:00:43 UTC (rev 212717)
</span><span class="lines">@@ -41,7 +41,7 @@
</span><span class="cx"> {
</span><span class="cx"> }
</span><span class="cx">
</span><del>-void JSModuleNamespaceObject::finishCreation(ExecState* exec, JSGlobalObject*, AbstractModuleRecord* moduleRecord, Vector<std::pair<Identifier, AbstractModuleRecord::Resolution>>&& resolutions)
</del><ins>+void JSModuleNamespaceObject::finishCreation(ExecState* exec, JSGlobalObject*, AbstractModuleRecord* moduleRecord, const IdentifierSet& exports)
</ins><span class="cx"> {
</span><span class="cx"> VM& vm = exec->vm();
</span><span class="cx"> auto scope = DECLARE_THROW_SCOPE(vm);
</span><span class="lines">@@ -54,25 +54,17 @@
</span><span class="cx"> // The list is ordered as if an Array of those String values had been sorted using Array.prototype.sort using SortCompare as comparefn.
</span><span class="cx"> //
</span><span class="cx"> // Sort the exported names by the code point order.
</span><del>- std::sort(resolutions.begin(), resolutions.end(), [] (const auto& lhs, const auto& rhs) {
- return codePointCompare(lhs.first.impl(), rhs.first.impl()) < 0;
</del><ins>+ Vector<UniquedStringImpl*> temporaryVector(exports.size(), nullptr);
+ std::transform(exports.begin(), exports.end(), temporaryVector.begin(), [](const RefPtr<WTF::UniquedStringImpl>& ref) {
+ return ref.get();
</ins><span class="cx"> });
</span><ins>+ std::sort(temporaryVector.begin(), temporaryVector.end(), [] (UniquedStringImpl* lhs, UniquedStringImpl* rhs) {
+ return codePointCompare(lhs, rhs) < 0;
+ });
+ for (auto* identifier : temporaryVector)
+ m_exports.add(identifier);
</ins><span class="cx">
</span><span class="cx"> m_moduleRecord.set(vm, this, moduleRecord);
</span><del>- {
- unsigned moduleRecordOffset = 0;
- m_names.reserveCapacity(resolutions.size());
- for (const auto& pair : resolutions) {
- moduleRecordAt(moduleRecordOffset).set(vm, this, pair.second.moduleRecord);
- m_names.append(pair.first);
- m_exports.add(pair.first.impl(), ExportEntry {
- pair.second.localName,
- moduleRecordOffset
- });
- ++moduleRecordOffset;
- }
- }
-
</del><span class="cx"> putDirect(vm, vm.propertyNames->toStringTagSymbol, jsString(&vm, "Module"), DontEnum | DontDelete | ReadOnly);
</span><span class="cx">
</span><span class="cx"> // http://www.ecma-international.org/ecma-262/6.0/#sec-module-namespace-exotic-objects-getprototypeof
</span><span class="lines">@@ -95,24 +87,8 @@
</span><span class="cx"> ASSERT_GC_OBJECT_INHERITS(thisObject, info());
</span><span class="cx"> Base::visitChildren(thisObject, visitor);
</span><span class="cx"> visitor.append(thisObject->m_moduleRecord);
</span><del>- for (unsigned i = 0; i < thisObject->m_names.size(); ++i)
- visitor.appendHidden(thisObject->moduleRecordAt(i));
</del><span class="cx"> }
</span><span class="cx">
</span><del>-static JSValue getValue(JSModuleEnvironment* environment, PropertyName localName, ScopeOffset& scopeOffset)
-{
- SymbolTable* symbolTable = environment->symbolTable();
- {
- ConcurrentJSLocker locker(symbolTable->m_lock);
- auto iter = symbolTable->find(locker, localName.uid());
- ASSERT(iter != symbolTable->end(locker));
- SymbolTableEntry& entry = iter->value;
- ASSERT(!entry.isNull());
- scopeOffset = entry.scopeOffset();
- }
- return environment->variableAt(scopeOffset).get();
-}
-
</del><span class="cx"> bool JSModuleNamespaceObject::getOwnPropertySlot(JSObject* cell, ExecState* exec, PropertyName propertyName, PropertySlot& slot)
</span><span class="cx"> {
</span><span class="cx"> VM& vm = exec->vm();
</span><span class="lines">@@ -128,19 +104,31 @@
</span><span class="cx"> if (propertyName.isSymbol())
</span><span class="cx"> return JSObject::getOwnPropertySlot(thisObject, exec, propertyName, slot);
</span><span class="cx">
</span><ins>+ // FIXME: Add IC for module namespace object.
+ // https://bugs.webkit.org/show_bug.cgi?id=160590
+ slot.disableCaching();
</ins><span class="cx"> slot.setIsTaintedByOpaqueObject();
</span><del>-
- auto iterator = thisObject->m_exports.find(propertyName.uid());
- if (iterator == thisObject->m_exports.end())
</del><ins>+ if (!thisObject->m_exports.contains(propertyName.uid()))
</ins><span class="cx"> return false;
</span><del>- ExportEntry& exportEntry = iterator->value;
</del><span class="cx">
</span><span class="cx"> switch (slot.internalMethodType()) {
</span><del>- case PropertySlot::InternalMethodType::GetOwnProperty:
- case PropertySlot::InternalMethodType::Get: {
- JSModuleEnvironment* environment = thisObject->moduleRecordAt(exportEntry.moduleRecordOffset)->moduleEnvironment();
- ScopeOffset scopeOffset;
- JSValue value = getValue(environment, exportEntry.localName, scopeOffset);
</del><ins>+ case PropertySlot::InternalMethodType::Get:
+ case PropertySlot::InternalMethodType::GetOwnProperty: {
+ AbstractModuleRecord* moduleRecord = thisObject->moduleRecord();
+
+ AbstractModuleRecord::Resolution resolution = moduleRecord->resolveExport(exec, Identifier::fromUid(exec, propertyName.uid()));
+ ASSERT(resolution.type != AbstractModuleRecord::Resolution::Type::NotFound && resolution.type != AbstractModuleRecord::Resolution::Type::Ambiguous);
+
+ AbstractModuleRecord* targetModule = resolution.moduleRecord;
+ JSModuleEnvironment* targetEnvironment = targetModule->moduleEnvironment();
+
+ PropertySlot trampolineSlot(targetEnvironment, PropertySlot::InternalMethodType::Get);
+ bool found = targetEnvironment->methodTable(vm)->getOwnPropertySlot(targetEnvironment, exec, resolution.localName, trampolineSlot);
+ ASSERT_UNUSED(found, found);
+
+ JSValue value = trampolineSlot.getValue(exec, propertyName);
+ ASSERT(!scope.exception());
+
</ins><span class="cx"> // If the value is filled with TDZ value, throw a reference error.
</span><span class="cx"> if (!value) {
</span><span class="cx"> throwVMError(exec, scope, createTDZError(exec));
</span><span class="lines">@@ -147,10 +135,9 @@
</span><span class="cx"> return false;
</span><span class="cx"> }
</span><span class="cx">
</span><del>- slot.setValueModuleNamespace(thisObject, DontDelete, value, environment, scopeOffset);
</del><ins>+ slot.setValue(thisObject, DontDelete, value);
</ins><span class="cx"> return true;
</span><span class="cx"> }
</span><del>-
</del><span class="cx"> case PropertySlot::InternalMethodType::HasProperty: {
</span><span class="cx"> // Do not perform [[Get]] for [[HasProperty]].
</span><span class="cx"> // [[Get]] / [[GetOwnProperty]] onto namespace object could throw an error while [[HasProperty]] just returns true here.
</span><span class="lines">@@ -202,8 +189,8 @@
</span><span class="cx"> {
</span><span class="cx"> // http://www.ecma-international.org/ecma-262/6.0/#sec-module-namespace-exotic-objects-ownpropertykeys
</span><span class="cx"> JSModuleNamespaceObject* thisObject = jsCast<JSModuleNamespaceObject*>(cell);
</span><del>- for (const auto& name : thisObject->m_names)
- propertyNames.add(name.impl());
</del><ins>+ for (const auto& name : thisObject->m_exports)
+ propertyNames.add(name.get());
</ins><span class="cx"> return JSObject::getOwnPropertyNames(thisObject, exec, propertyNames, mode);
</span><span class="cx"> }
</span><span class="cx">
</span></span></pre></div>
<a id="trunkSourceJavaScriptCoreruntimeJSModuleNamespaceObjecth"></a>
<div class="modfile"><h4>Modified: trunk/Source/JavaScriptCore/runtime/JSModuleNamespaceObject.h (212716 => 212717)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/JavaScriptCore/runtime/JSModuleNamespaceObject.h 2017-02-21 17:25:02 UTC (rev 212716)
+++ trunk/Source/JavaScriptCore/runtime/JSModuleNamespaceObject.h 2017-02-21 18:00:43 UTC (rev 212717)
</span><span class="lines">@@ -25,25 +25,22 @@
</span><span class="cx">
</span><span class="cx"> #pragma once
</span><span class="cx">
</span><del>-#include "AbstractModuleRecord.h"
</del><span class="cx"> #include "JSDestructibleObject.h"
</span><del>-#include "ScopeOffset.h"
</del><ins>+#include <wtf/ListHashSet.h>
</ins><span class="cx">
</span><span class="cx"> namespace JSC {
</span><span class="cx">
</span><del>-class JSModuleNamespaceObject final : public JSDestructibleObject {
</del><ins>+class AbstractModuleRecord;
+
+class JSModuleNamespaceObject : public JSDestructibleObject {
</ins><span class="cx"> public:
</span><span class="cx"> typedef JSDestructibleObject Base;
</span><span class="cx"> static const unsigned StructureFlags = Base::StructureFlags | OverridesGetOwnPropertySlot | InterceptsGetOwnPropertySlotByIndexEvenWhenLengthIsNotZero | OverridesGetPropertyNames | GetOwnPropertySlotIsImpureForPropertyAbsence | IsImmutablePrototypeExoticObject;
</span><span class="cx">
</span><del>- static JSModuleNamespaceObject* create(ExecState* exec, JSGlobalObject* globalObject, Structure* structure, AbstractModuleRecord* moduleRecord, Vector<std::pair<Identifier, AbstractModuleRecord::Resolution>>&& resolutions)
</del><ins>+ static JSModuleNamespaceObject* create(ExecState* exec, JSGlobalObject* globalObject, Structure* structure, AbstractModuleRecord* moduleRecord, const IdentifierSet& exports)
</ins><span class="cx"> {
</span><del>- JSModuleNamespaceObject* object =
- new (
- NotNull,
- allocateCell<JSModuleNamespaceObject>(exec->vm().heap, JSModuleNamespaceObject::allocationSize(resolutions.size())))
- JSModuleNamespaceObject(exec->vm(), structure);
- object->finishCreation(exec, globalObject, moduleRecord, WTFMove(resolutions));
</del><ins>+ JSModuleNamespaceObject* object = new (NotNull, allocateCell<JSModuleNamespaceObject>(exec->vm().heap)) JSModuleNamespaceObject(exec->vm(), structure);
+ object->finishCreation(exec, globalObject, moduleRecord, exports);
</ins><span class="cx"> return object;
</span><span class="cx"> }
</span><span class="cx">
</span><span class="lines">@@ -64,7 +61,7 @@
</span><span class="cx"> AbstractModuleRecord* moduleRecord() { return m_moduleRecord.get(); }
</span><span class="cx">
</span><span class="cx"> protected:
</span><del>- JS_EXPORT_PRIVATE void finishCreation(ExecState*, JSGlobalObject*, AbstractModuleRecord*, Vector<std::pair<Identifier, AbstractModuleRecord::Resolution>>&&);
</del><ins>+ JS_EXPORT_PRIVATE void finishCreation(ExecState*, JSGlobalObject*, AbstractModuleRecord*, const IdentifierSet& exports);
</ins><span class="cx"> JS_EXPORT_PRIVATE JSModuleNamespaceObject(VM&, Structure*);
</span><span class="cx">
</span><span class="cx"> private:
</span><span class="lines">@@ -71,43 +68,10 @@
</span><span class="cx"> static void destroy(JSCell*);
</span><span class="cx"> static void visitChildren(JSCell*, SlotVisitor&);
</span><span class="cx">
</span><del>- WriteBarrierBase<AbstractModuleRecord>& moduleRecordAt(unsigned offset)
- {
- return moduleRecords()[offset];
- }
</del><ins>+ typedef WTF::ListHashSet<RefPtr<UniquedStringImpl>, IdentifierRepHash> OrderedIdentifierSet;
</ins><span class="cx">
</span><del>- WriteBarrierBase<AbstractModuleRecord>* moduleRecords()
- {
- return bitwise_cast<WriteBarrierBase<AbstractModuleRecord>*>(bitwise_cast<char*>(this) + offsetOfModuleRecords());
- }
-
- static size_t offsetOfModuleRecords()
- {
- return WTF::roundUpToMultipleOf<sizeof(WriteBarrier<AbstractModuleRecord>)>(sizeof(JSModuleNamespaceObject));
- }
-
- static size_t allocationSize(unsigned moduleRecords)
- {
- return offsetOfModuleRecords() + moduleRecords * sizeof(WriteBarrier<AbstractModuleRecord>);
- }
-
- struct ExportEntry {
- Identifier localName;
- unsigned moduleRecordOffset;
- };
-
- typedef HashMap<RefPtr<UniquedStringImpl>, ExportEntry, IdentifierRepHash, HashTraits<RefPtr<UniquedStringImpl>>> ExportMap;
-
- ExportMap m_exports;
- Vector<Identifier> m_names;
</del><ins>+ OrderedIdentifierSet m_exports;
</ins><span class="cx"> WriteBarrier<AbstractModuleRecord> m_moduleRecord;
</span><span class="cx"> };
</span><span class="cx">
</span><del>-inline bool isJSModuleNamespaceObject(JSCell* cell)
-{
- return cell->classInfo(*cell->vm()) == JSModuleNamespaceObject::info();
-}
-
-inline bool isJSModuleNamespaceObject(JSValue v) { return v.isCell() && isJSModuleNamespaceObject(v.asCell()); }
-
</del><span class="cx"> } // namespace JSC
</span></span></pre></div>
<a id="trunkSourceJavaScriptCoreruntimeJSModuleRecordh"></a>
<div class="modfile"><h4>Modified: trunk/Source/JavaScriptCore/runtime/JSModuleRecord.h (212716 => 212717)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/JavaScriptCore/runtime/JSModuleRecord.h 2017-02-21 17:25:02 UTC (rev 212716)
+++ trunk/Source/JavaScriptCore/runtime/JSModuleRecord.h 2017-02-21 18:00:43 UTC (rev 212717)
</span><span class="lines">@@ -40,6 +40,11 @@
</span><span class="cx">
</span><span class="cx"> DECLARE_EXPORT_INFO;
</span><span class="cx">
</span><ins>+ JSModuleEnvironment* moduleEnvironment()
+ {
+ ASSERT(m_moduleEnvironment);
+ return m_moduleEnvironment.get();
+ }
</ins><span class="cx"> static Structure* createStructure(VM&, JSGlobalObject*, JSValue);
</span><span class="cx"> static JSModuleRecord* create(ExecState*, VM&, Structure*, const Identifier&, const SourceCode&, const VariableEnvironment&, const VariableEnvironment&);
</span><span class="cx">
</span></span></pre></div>
<a id="trunkSourceJavaScriptCoreruntimePropertySloth"></a>
<div class="modfile"><h4>Modified: trunk/Source/JavaScriptCore/runtime/PropertySlot.h (212716 => 212717)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/JavaScriptCore/runtime/PropertySlot.h 2017-02-21 17:25:02 UTC (rev 212716)
+++ trunk/Source/JavaScriptCore/runtime/PropertySlot.h 2017-02-21 18:00:43 UTC (rev 212717)
</span><span class="lines">@@ -23,7 +23,6 @@
</span><span class="cx"> #include "JSCJSValue.h"
</span><span class="cx"> #include "PropertyName.h"
</span><span class="cx"> #include "PropertyOffset.h"
</span><del>-#include "ScopeOffset.h"
</del><span class="cx"> #include <wtf/Assertions.h>
</span><span class="cx">
</span><span class="cx"> namespace JSC {
</span><span class="lines">@@ -33,7 +32,6 @@
</span><span class="cx"> class ExecState;
</span><span class="cx"> class GetterSetter;
</span><span class="cx"> class JSObject;
</span><del>-class JSModuleEnvironment;
</del><span class="cx">
</span><span class="cx"> // ECMA 262-3 8.6.1
</span><span class="cx"> // Property attributes
</span><span class="lines">@@ -88,12 +86,6 @@
</span><span class="cx"> VMInquiry, // Our VM is just poking around. When this is the InternalMethodType, getOwnPropertySlot is not allowed to do user observable actions.
</span><span class="cx"> };
</span><span class="cx">
</span><del>- enum class AdditionalDataType : uint8_t {
- None,
- DOMJIT, // Annotated with DOMJIT information.
- ModuleNamespace, // ModuleNamespaceObject's environment access.
- };
-
</del><span class="cx"> explicit PropertySlot(const JSValue thisValue, InternalMethodType internalMethodType)
</span><span class="cx"> : m_offset(invalidOffset)
</span><span class="cx"> , m_thisValue(thisValue)
</span><span class="lines">@@ -102,7 +94,6 @@
</span><span class="cx"> , m_cacheability(CachingAllowed)
</span><span class="cx"> , m_propertyType(TypeUnset)
</span><span class="cx"> , m_internalMethodType(internalMethodType)
</span><del>- , m_additionalDataType(AdditionalDataType::None)
</del><span class="cx"> , m_isTaintedByOpaqueObject(false)
</span><span class="cx"> {
</span><span class="cx"> }
</span><span class="lines">@@ -172,23 +163,9 @@
</span><span class="cx">
</span><span class="cx"> DOMJIT::GetterSetter* domJIT() const
</span><span class="cx"> {
</span><del>- if (m_additionalDataType == AdditionalDataType::DOMJIT)
- return m_additionalData.domJIT;
- return nullptr;
</del><ins>+ return m_domJIT;
</ins><span class="cx"> }
</span><span class="cx">
</span><del>- struct ModuleNamespaceSlot {
- JSModuleEnvironment* environment;
- unsigned scopeOffset;
- };
-
- std::optional<ModuleNamespaceSlot> moduleNamespaceSlot() const
- {
- if (m_additionalDataType == AdditionalDataType::ModuleNamespace)
- return m_additionalData.moduleNamespaceSlot;
- return std::nullopt;
- }
-
</del><span class="cx"> void setValue(JSObject* slotBase, unsigned attributes, JSValue value)
</span><span class="cx"> {
</span><span class="cx"> ASSERT(attributes == attributesForStructure(attributes));
</span><span class="lines">@@ -229,14 +206,6 @@
</span><span class="cx"> m_offset = invalidOffset;
</span><span class="cx"> }
</span><span class="cx">
</span><del>- void setValueModuleNamespace(JSObject* slotBase, unsigned attributes, JSValue value, JSModuleEnvironment* environment, ScopeOffset scopeOffset)
- {
- setValue(slotBase, attributes, value);
- m_additionalDataType = AdditionalDataType::ModuleNamespace;
- m_additionalData.moduleNamespaceSlot.environment = environment;
- m_additionalData.moduleNamespaceSlot.scopeOffset = scopeOffset.offset();
- }
-
</del><span class="cx"> void setCustom(JSObject* slotBase, unsigned attributes, GetValueFunc getValue)
</span><span class="cx"> {
</span><span class="cx"> ASSERT(attributes == attributesForStructure(attributes));
</span><span class="lines">@@ -251,7 +220,7 @@
</span><span class="cx"> m_offset = invalidOffset;
</span><span class="cx"> }
</span><span class="cx">
</span><del>- void setCacheableCustom(JSObject* slotBase, unsigned attributes, GetValueFunc getValue)
</del><ins>+ void setCacheableCustom(JSObject* slotBase, unsigned attributes, GetValueFunc getValue, DOMJIT::GetterSetter* domJIT = nullptr)
</ins><span class="cx"> {
</span><span class="cx"> ASSERT(attributes == attributesForStructure(attributes));
</span><span class="cx">
</span><span class="lines">@@ -263,17 +232,9 @@
</span><span class="cx"> m_slotBase = slotBase;
</span><span class="cx"> m_propertyType = TypeCustom;
</span><span class="cx"> m_offset = !invalidOffset;
</span><ins>+ m_domJIT = domJIT;
</ins><span class="cx"> }
</span><span class="cx">
</span><del>- void setCacheableCustom(JSObject* slotBase, unsigned attributes, GetValueFunc getValue, DOMJIT::GetterSetter* domJIT)
- {
- setCacheableCustom(slotBase, attributes, getValue);
- if (domJIT) {
- m_additionalDataType = AdditionalDataType::DOMJIT;
- m_additionalData.domJIT = domJIT;
- }
- }
-
</del><span class="cx"> void setCustomGetterSetter(JSObject* slotBase, unsigned attributes, CustomGetterSetter* getterSetter)
</span><span class="cx"> {
</span><span class="cx"> ASSERT(attributes == attributesForStructure(attributes));
</span><span class="lines">@@ -367,11 +328,7 @@
</span><span class="cx"> CacheabilityType m_cacheability;
</span><span class="cx"> PropertyType m_propertyType;
</span><span class="cx"> InternalMethodType m_internalMethodType;
</span><del>- AdditionalDataType m_additionalDataType;
- union {
- DOMJIT::GetterSetter* domJIT;
- ModuleNamespaceSlot moduleNamespaceSlot;
- } m_additionalData;
</del><ins>+ DOMJIT::GetterSetter* m_domJIT { nullptr };
</ins><span class="cx"> bool m_isTaintedByOpaqueObject;
</span><span class="cx"> };
</span><span class="cx">
</span></span></pre>
</div>
</div>
</body>
</html>