<!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>[183557] trunk/Source/JavaScriptCore</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/183557">183557</a></dd>

<dt>Author</dt> <dd>fpizlo@apple.com</dd>

<dt>Date</dt> <dd>2015-04-29 10:38:28 -0700 (Wed, 29 Apr 2015)</dd>

</dl>

<h3>Log Message</h3>

<pre>Evict IsEnvironmentRecord from inline type flags

https://bugs.webkit.org/show_bug.cgi?id=144398

Reviewed by Mark Lam and Michael Saboff.

In https://bugs.webkit.org/show_bug.cgi?id=144397, we'll need an extra bit in the inline

type flags. This change picks the least important inline type flag - IsEnvironmentRecord -

and evicts it into the out-of-line type flags. This change has no performance implications

because we never even accessed IsEnvironmentRecord via the StructureIDBlob. The only place

where we access it at all is in String.prototype.repeat, and there we already load the

structure anyway.

* runtime/JSTypeInfo.h:

(JSC::TypeInfo::implementsHasInstance):

(JSC::TypeInfo::structureIsImmortal):

(JSC::TypeInfo::isEnvironmentRecord):</pre>

<h3>Modified Paths</h3>

<ul>

<li><a href="#trunkSourceJavaScriptCoreChangeLog">trunk/Source/JavaScriptCore/ChangeLog</a></li>

<li><a href="#trunkSourceJavaScriptCoreruntimeJSTypeInfoh">trunk/Source/JavaScriptCore/runtime/JSTypeInfo.h</a></li>

</ul>

</div>

<div id="patch">

<h3>Diff</h3>

<a id="trunkSourceJavaScriptCoreChangeLog"></a>

<div class="modfile"><h4>Modified: trunk/Source/JavaScriptCore/ChangeLog (183556 => 183557)</h4>

<pre class="diff"><span>

<span class="info">--- trunk/Source/JavaScriptCore/ChangeLog        2015-04-29 17:31:19 UTC (rev 183556)

+++ trunk/Source/JavaScriptCore/ChangeLog        2015-04-29 17:38:28 UTC (rev 183557)

</span><span class="lines">@@ -1,3 +1,22 @@

</span><ins>+2015-04-29  Filip Pizlo  &lt;fpizlo@apple.com&gt;

+

+        Evict IsEnvironmentRecord from inline type flags

+        https://bugs.webkit.org/show_bug.cgi?id=144398

+

+        Reviewed by Mark Lam and Michael Saboff.

+        

+        In https://bugs.webkit.org/show_bug.cgi?id=144397, we'll need an extra bit in the inline

+        type flags. This change picks the least important inline type flag - IsEnvironmentRecord -

+        and evicts it into the out-of-line type flags. This change has no performance implications

+        because we never even accessed IsEnvironmentRecord via the StructureIDBlob. The only place

+        where we access it at all is in String.prototype.repeat, and there we already load the

+        structure anyway.

+

+        * runtime/JSTypeInfo.h:

+        (JSC::TypeInfo::implementsHasInstance):

+        (JSC::TypeInfo::structureIsImmortal):

+        (JSC::TypeInfo::isEnvironmentRecord):

+

</ins><span class="cx"> 2015-04-29  Darin Adler  &lt;darin@apple.com&gt;

</span><span class="cx"> 

</span><span class="cx">         [ES6] Implement Unicode code point escapes

</span></span></pre></div>

<a id="trunkSourceJavaScriptCoreruntimeJSTypeInfoh"></a>

<div class="modfile"><h4>Modified: trunk/Source/JavaScriptCore/runtime/JSTypeInfo.h (183556 => 183557)</h4>

<pre class="diff"><span>

<span class="info">--- trunk/Source/JavaScriptCore/runtime/JSTypeInfo.h        2015-04-29 17:31:19 UTC (rev 183556)

+++ trunk/Source/JavaScriptCore/runtime/JSTypeInfo.h        2015-04-29 17:38:28 UTC (rev 183557)

</span><span class="lines">@@ -1,6 +1,6 @@

</span><span class="cx"> // -*- mode: c++; c-basic-offset: 4 -*-

</span><span class="cx"> /*

</del><ins>+ * Copyright (C) 2008, 2015 Apple Inc. All rights reserved.

</ins><span class="cx">  *

</span><span class="cx">  * Redistribution and use in source and binary forms, with or without

</span><span class="cx">  * modification, are permitted provided that the following conditions

</span><span class="lines">@@ -40,7 +40,6 @@

</span><span class="cx"> static const unsigned ImplementsHasInstance = 1 &lt;&lt; 1;

</span><span class="cx"> static const unsigned OverridesHasInstance = 1 &lt;&lt; 2;

</span><span class="cx"> static const unsigned ImplementsDefaultHasInstance = 1 &lt;&lt; 3;

</span><del>-static const unsigned IsEnvironmentRecord = 1 &lt;&lt; 4;

</del><span class="cx"> static const unsigned OverridesGetOwnPropertySlot = 1 &lt;&lt; 5;

</span><span class="cx"> static const unsigned InterceptsGetOwnPropertySlotByIndexEvenWhenLengthIsNotZero = 1 &lt;&lt; 6;

</span><span class="cx"> static const unsigned StructureIsImmortal = 1 &lt;&lt; 7;

</span><span class="lines">@@ -49,6 +48,7 @@

</span><span class="cx"> static const unsigned ProhibitsPropertyCaching = 1 &lt;&lt; 9;

</span><span class="cx"> static const unsigned HasImpureGetOwnPropertySlot = 1 &lt;&lt; 10;

</span><span class="cx"> static const unsigned NewImpurePropertyFiresWatchpoints = 1 &lt;&lt; 11;

</span><ins>+static const unsigned IsEnvironmentRecord = 1 &lt;&lt; 12;

</ins><span class="cx"> 

</span><span class="cx"> class TypeInfo {

</span><span class="cx"> public:

</span><span class="lines">@@ -81,17 +81,17 @@

</span><span class="cx">     unsigned flags() const { return (static_cast&lt;unsigned&gt;(m_flags2) &lt;&lt; 8) | static_cast&lt;unsigned&gt;(m_flags); }

</span><span class="cx">     bool masqueradesAsUndefined() const { return isSetOnFlags1(MasqueradesAsUndefined); }

</span><span class="cx">     bool implementsHasInstance() const { return isSetOnFlags1(ImplementsHasInstance); }

</span><del>-    bool isEnvironmentRecord() const { return isSetOnFlags1(IsEnvironmentRecord); }

</del><span class="cx">     bool overridesHasInstance() const { return isSetOnFlags1(OverridesHasInstance); }

</span><span class="cx">     bool implementsDefaultHasInstance() const { return isSetOnFlags1(ImplementsDefaultHasInstance); }

</span><span class="cx">     bool overridesGetOwnPropertySlot() const { return overridesGetOwnPropertySlot(inlineTypeFlags()); }

</span><span class="cx">     static bool overridesGetOwnPropertySlot(InlineTypeFlags flags) { return flags &amp; OverridesGetOwnPropertySlot; }

</span><span class="cx">     bool interceptsGetOwnPropertySlotByIndexEvenWhenLengthIsNotZero() const { return isSetOnFlags1(InterceptsGetOwnPropertySlotByIndexEvenWhenLengthIsNotZero); }

</span><ins>+    bool structureIsImmortal() const { return isSetOnFlags1(StructureIsImmortal); }

</ins><span class="cx">     bool overridesGetPropertyNames() const { return isSetOnFlags2(OverridesGetPropertyNames); }

</span><span class="cx">     bool prohibitsPropertyCaching() const { return isSetOnFlags2(ProhibitsPropertyCaching); }

</span><span class="cx">     bool hasImpureGetOwnPropertySlot() const { return isSetOnFlags2(HasImpureGetOwnPropertySlot); }

</span><span class="cx">     bool newImpurePropertyFiresWatchpoints() const { return isSetOnFlags2(NewImpurePropertyFiresWatchpoints); }

</span><del>-    bool structureIsImmortal() const { return isSetOnFlags1(StructureIsImmortal); }

</del><ins>+    bool isEnvironmentRecord() const { return isSetOnFlags2(IsEnvironmentRecord); }

</ins><span class="cx"> 

</span><span class="cx">     static ptrdiff_t flagsOffset()

</span><span class="cx">     {

</span></span></pre>

</div>

</div>

</body>

</html>