<!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>[209795] trunk/Source</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/209795">209795</a></dd>
<dt>Author</dt> <dd>barraclough@apple.com</dd>
<dt>Date</dt> <dd>2016-12-14 01:35:18 -0800 (Wed, 14 Dec 2016)</dd>
</dl>
<h3>Log Message</h3>
<pre>MarkedBlock::marksConveyLivenessDuringMarking should take into account collection scope
https://bugs.webkit.org/show_bug.cgi?id=165741
Unreviewed rollout due to performance regression.
Source/JavaScriptCore:
* CMakeLists.txt:
* JavaScriptCore.xcodeproj/project.pbxproj:
* heap/CellContainer.cpp: Removed.
* heap/CellContainer.h:
* heap/MarkedAllocator.cpp:
(JSC::MarkedAllocator::addBlock):
(JSC::MarkedAllocator::removeBlock):
(JSC::MarkedAllocator::dumpBits):
* heap/MarkedAllocator.h:
(JSC::MarkedAllocator::forEachBitVector):
(JSC::MarkedAllocator::forEachBitVectorWithName):
* heap/MarkedBlock.cpp:
(JSC::MarkedBlock::tryCreate):
(JSC::MarkedBlock::Handle::~Handle):
(JSC::MarkedBlock::MarkedBlock):
(JSC::MarkedBlock::Handle::specializedSweep):
(JSC::MarkedBlock::Handle::sweepHelperSelectMarksMode):
(JSC::MarkedBlock::Handle::stopAllocating):
(JSC::MarkedBlock::Handle::resumeAllocating):
(JSC::MarkedBlock::aboutToMarkSlow):
(JSC::MarkedBlock::Handle::didConsumeFreeList):
(JSC::MarkedBlock::Handle::dumpState): Deleted.
* heap/MarkedBlock.h:
(JSC::MarkedBlock::isMarked):
(JSC::MarkedBlock::markingVersion): Deleted.
(JSC::MarkedBlock::isMarkedRaw): Deleted.
* heap/MarkedBlockInlines.h:
(JSC::MarkedBlock::marksConveyLivenessDuringMarking):
* heap/SlotVisitor.cpp:
(JSC::SlotVisitor::appendJSCellOrAuxiliary):
* runtime/Options.h:
* runtime/StructureIDTable.h:
(JSC::StructureIDTable::get):
(JSC::StructureIDTable::size): Deleted.
Source/WTF:
* WTF.xcodeproj/project.pbxproj:
* wtf/CMakeLists.txt:
* wtf/DataLog.cpp:
(WTF::initializeLogFileOnce):
(WTF::initializeLogFile):
(WTF::dataFile):
* wtf/DataLog.h:
* wtf/LockedPrintStream.cpp: Removed.
* wtf/LockedPrintStream.h: Removed.
* wtf/PrintStream.cpp:
(WTF::PrintStream::printfVariableFormat): Deleted.
(WTF::PrintStream::begin): Deleted.
(WTF::PrintStream::end): Deleted.
* wtf/PrintStream.h:
(WTF::PrintStream::print):
(WTF::PrintStream::println):
(WTF::PrintStream::atomically): Deleted.
(WTF::PrintStream::printImpl): Deleted.
(): Deleted.
(WTF::>::unpack): Deleted.
(WTF::FormatImpl::FormatImpl): Deleted.
(WTF::FormatImpl::dump): Deleted.
(WTF::format): Deleted.
* wtf/RecursiveLockAdapter.h: Removed.
* wtf/WordLock.cpp:
* wtf/WordLock.h:</pre>
<h3>Modified Paths</h3>
<ul>
<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="#trunkSourceJavaScriptCoreheapCellContainerh">trunk/Source/JavaScriptCore/heap/CellContainer.h</a></li>
<li><a href="#trunkSourceJavaScriptCoreheapMarkedAllocatorcpp">trunk/Source/JavaScriptCore/heap/MarkedAllocator.cpp</a></li>
<li><a href="#trunkSourceJavaScriptCoreheapMarkedAllocatorh">trunk/Source/JavaScriptCore/heap/MarkedAllocator.h</a></li>
<li><a href="#trunkSourceJavaScriptCoreheapMarkedBlockcpp">trunk/Source/JavaScriptCore/heap/MarkedBlock.cpp</a></li>
<li><a href="#trunkSourceJavaScriptCoreheapMarkedBlockh">trunk/Source/JavaScriptCore/heap/MarkedBlock.h</a></li>
<li><a href="#trunkSourceJavaScriptCoreheapMarkedBlockInlinesh">trunk/Source/JavaScriptCore/heap/MarkedBlockInlines.h</a></li>
<li><a href="#trunkSourceJavaScriptCoreheapSlotVisitorcpp">trunk/Source/JavaScriptCore/heap/SlotVisitor.cpp</a></li>
<li><a href="#trunkSourceJavaScriptCoreruntimeOptionsh">trunk/Source/JavaScriptCore/runtime/Options.h</a></li>
<li><a href="#trunkSourceJavaScriptCoreruntimeStructureIDTableh">trunk/Source/JavaScriptCore/runtime/StructureIDTable.h</a></li>
<li><a href="#trunkSourceWTFChangeLog">trunk/Source/WTF/ChangeLog</a></li>
<li><a href="#trunkSourceWTFWTFxcodeprojprojectpbxproj">trunk/Source/WTF/WTF.xcodeproj/project.pbxproj</a></li>
<li><a href="#trunkSourceWTFwtfCMakeListstxt">trunk/Source/WTF/wtf/CMakeLists.txt</a></li>
<li><a href="#trunkSourceWTFwtfDataLogcpp">trunk/Source/WTF/wtf/DataLog.cpp</a></li>
<li><a href="#trunkSourceWTFwtfDataLogh">trunk/Source/WTF/wtf/DataLog.h</a></li>
<li><a href="#trunkSourceWTFwtfPrintStreamcpp">trunk/Source/WTF/wtf/PrintStream.cpp</a></li>
<li><a href="#trunkSourceWTFwtfPrintStreamh">trunk/Source/WTF/wtf/PrintStream.h</a></li>
<li><a href="#trunkSourceWTFwtfWordLockcpp">trunk/Source/WTF/wtf/WordLock.cpp</a></li>
<li><a href="#trunkSourceWTFwtfWordLockh">trunk/Source/WTF/wtf/WordLock.h</a></li>
</ul>
<h3>Removed Paths</h3>
<ul>
<li><a href="#trunkSourceJavaScriptCoreheapCellContainercpp">trunk/Source/JavaScriptCore/heap/CellContainer.cpp</a></li>
<li><a href="#trunkSourceWTFwtfLockedPrintStreamcpp">trunk/Source/WTF/wtf/LockedPrintStream.cpp</a></li>
<li><a href="#trunkSourceWTFwtfLockedPrintStreamh">trunk/Source/WTF/wtf/LockedPrintStream.h</a></li>
<li><a href="#trunkSourceWTFwtfRecursiveLockAdapterh">trunk/Source/WTF/wtf/RecursiveLockAdapter.h</a></li>
</ul>
</div>
<div id="patch">
<h3>Diff</h3>
<a id="trunkSourceJavaScriptCoreCMakeListstxt"></a>
<div class="modfile"><h4>Modified: trunk/Source/JavaScriptCore/CMakeLists.txt (209794 => 209795)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/JavaScriptCore/CMakeLists.txt 2016-12-14 08:23:23 UTC (rev 209794)
+++ trunk/Source/JavaScriptCore/CMakeLists.txt 2016-12-14 09:35:18 UTC (rev 209795)
</span><span class="lines">@@ -464,7 +464,6 @@
</span><span class="cx"> ftl/FTLValueRange.cpp
</span><span class="cx">
</span><span class="cx"> heap/AllocatorAttributes.cpp
</span><del>- heap/CellContainer.cpp
</del><span class="cx"> heap/CodeBlockSet.cpp
</span><span class="cx"> heap/CollectionScope.cpp
</span><span class="cx"> heap/ConservativeRoots.cpp
</span></span></pre></div>
<a id="trunkSourceJavaScriptCoreChangeLog"></a>
<div class="modfile"><h4>Modified: trunk/Source/JavaScriptCore/ChangeLog (209794 => 209795)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/JavaScriptCore/ChangeLog 2016-12-14 08:23:23 UTC (rev 209794)
+++ trunk/Source/JavaScriptCore/ChangeLog 2016-12-14 09:35:18 UTC (rev 209795)
</span><span class="lines">@@ -1,3 +1,45 @@
</span><ins>+2016-12-14 Gavin Barraclough <barraclough@apple.com>
+
+ MarkedBlock::marksConveyLivenessDuringMarking should take into account collection scope
+ https://bugs.webkit.org/show_bug.cgi?id=165741
+
+ Unreviewed rollout due to performance regression.
+
+ * CMakeLists.txt:
+ * JavaScriptCore.xcodeproj/project.pbxproj:
+ * heap/CellContainer.cpp: Removed.
+ * heap/CellContainer.h:
+ * heap/MarkedAllocator.cpp:
+ (JSC::MarkedAllocator::addBlock):
+ (JSC::MarkedAllocator::removeBlock):
+ (JSC::MarkedAllocator::dumpBits):
+ * heap/MarkedAllocator.h:
+ (JSC::MarkedAllocator::forEachBitVector):
+ (JSC::MarkedAllocator::forEachBitVectorWithName):
+ * heap/MarkedBlock.cpp:
+ (JSC::MarkedBlock::tryCreate):
+ (JSC::MarkedBlock::Handle::~Handle):
+ (JSC::MarkedBlock::MarkedBlock):
+ (JSC::MarkedBlock::Handle::specializedSweep):
+ (JSC::MarkedBlock::Handle::sweepHelperSelectMarksMode):
+ (JSC::MarkedBlock::Handle::stopAllocating):
+ (JSC::MarkedBlock::Handle::resumeAllocating):
+ (JSC::MarkedBlock::aboutToMarkSlow):
+ (JSC::MarkedBlock::Handle::didConsumeFreeList):
+ (JSC::MarkedBlock::Handle::dumpState): Deleted.
+ * heap/MarkedBlock.h:
+ (JSC::MarkedBlock::isMarked):
+ (JSC::MarkedBlock::markingVersion): Deleted.
+ (JSC::MarkedBlock::isMarkedRaw): Deleted.
+ * heap/MarkedBlockInlines.h:
+ (JSC::MarkedBlock::marksConveyLivenessDuringMarking):
+ * heap/SlotVisitor.cpp:
+ (JSC::SlotVisitor::appendJSCellOrAuxiliary):
+ * runtime/Options.h:
+ * runtime/StructureIDTable.h:
+ (JSC::StructureIDTable::get):
+ (JSC::StructureIDTable::size): Deleted.
+
</ins><span class="cx"> 2016-12-13 Commit Queue <commit-queue@webkit.org>
</span><span class="cx">
</span><span class="cx"> Unreviewed, rolling out r209792.
</span></span></pre></div>
<a id="trunkSourceJavaScriptCoreJavaScriptCorexcodeprojprojectpbxproj"></a>
<div class="modfile"><h4>Modified: trunk/Source/JavaScriptCore/JavaScriptCore.xcodeproj/project.pbxproj (209794 => 209795)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/JavaScriptCore/JavaScriptCore.xcodeproj/project.pbxproj 2016-12-14 08:23:23 UTC (rev 209794)
+++ trunk/Source/JavaScriptCore/JavaScriptCore.xcodeproj/project.pbxproj 2016-12-14 09:35:18 UTC (rev 209795)
</span><span class="lines">@@ -752,7 +752,6 @@
</span><span class="cx"> 0FDB2CEA174896C7007B3C1B /* ConcurrentJSLock.h in Headers */ = {isa = PBXBuildFile; fileRef = 0FDB2CE9174896C7007B3C1B /* ConcurrentJSLock.h */; settings = {ATTRIBUTES = (Private, ); }; };
</span><span class="cx"> 0FDDBFB51666EED800C55FEF /* DFGVariableAccessDataDump.cpp in Sources */ = {isa = PBXBuildFile; fileRef = 0FDDBFB21666EED500C55FEF /* DFGVariableAccessDataDump.cpp */; };
</span><span class="cx"> 0FDDBFB61666EEDA00C55FEF /* DFGVariableAccessDataDump.h in Headers */ = {isa = PBXBuildFile; fileRef = 0FDDBFB31666EED500C55FEF /* DFGVariableAccessDataDump.h */; };
</span><del>- 0FDE87F91DFD0C760064C390 /* CellContainer.cpp in Sources */ = {isa = PBXBuildFile; fileRef = 0FDE87F81DFD0C6D0064C390 /* CellContainer.cpp */; };
</del><span class="cx"> 0FDE87FC1DFE6E510064C390 /* SpaceTimeScheduler.h in Headers */ = {isa = PBXBuildFile; fileRef = 0FDE87FB1DFE6E500064C390 /* SpaceTimeScheduler.h */; };
</span><span class="cx"> 0FDE87FD1DFE6E540064C390 /* SpaceTimeScheduler.cpp in Sources */ = {isa = PBXBuildFile; fileRef = 0FDE87FA1DFE6E500064C390 /* SpaceTimeScheduler.cpp */; };
</span><span class="cx"> 0FDF67D21D9C6D27001B9825 /* B3Kind.h in Headers */ = {isa = PBXBuildFile; fileRef = 0FDF67D11D9C6086001B9825 /* B3Kind.h */; };
</span><span class="lines">@@ -3182,7 +3181,6 @@
</span><span class="cx"> 0FDB2CE9174896C7007B3C1B /* ConcurrentJSLock.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = ConcurrentJSLock.h; sourceTree = "<group>"; };
</span><span class="cx"> 0FDDBFB21666EED500C55FEF /* DFGVariableAccessDataDump.cpp */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp; name = DFGVariableAccessDataDump.cpp; path = dfg/DFGVariableAccessDataDump.cpp; sourceTree = "<group>"; };
</span><span class="cx"> 0FDDBFB31666EED500C55FEF /* DFGVariableAccessDataDump.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; name = DFGVariableAccessDataDump.h; path = dfg/DFGVariableAccessDataDump.h; sourceTree = "<group>"; };
</span><del>- 0FDE87F81DFD0C6D0064C390 /* CellContainer.cpp */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp; path = CellContainer.cpp; sourceTree = "<group>"; };
</del><span class="cx"> 0FDE87FA1DFE6E500064C390 /* SpaceTimeScheduler.cpp */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp; path = SpaceTimeScheduler.cpp; sourceTree = "<group>"; };
</span><span class="cx"> 0FDE87FB1DFE6E500064C390 /* SpaceTimeScheduler.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = SpaceTimeScheduler.h; sourceTree = "<group>"; };
</span><span class="cx"> 0FDF67D01D9C6086001B9825 /* B3Kind.cpp */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp; name = B3Kind.cpp; path = b3/B3Kind.cpp; sourceTree = "<group>"; };
</span><span class="lines">@@ -5637,7 +5635,6 @@
</span><span class="cx"> 0FA7620A1DB959F600B7A2FD /* AllocatingScope.h */,
</span><span class="cx"> 0F9630351D4192C3005609D9 /* AllocatorAttributes.cpp */,
</span><span class="cx"> 0F9630361D4192C3005609D9 /* AllocatorAttributes.h */,
</span><del>- 0FDE87F81DFD0C6D0064C390 /* CellContainer.cpp */,
</del><span class="cx"> 0F070A421D543A89006E7232 /* CellContainer.h */,
</span><span class="cx"> 0F070A431D543A89006E7232 /* CellContainerInlines.h */,
</span><span class="cx"> 0F1C3DD91BBCE09E00E523E4 /* CellState.h */,
</span><span class="lines">@@ -10257,7 +10254,6 @@
</span><span class="cx"> 0FB1058B1675483100F8AB6E /* ProfilerOSRExit.cpp in Sources */,
</span><span class="cx"> 0FB1058D1675483700F8AB6E /* ProfilerOSRExitSite.cpp in Sources */,
</span><span class="cx"> 0F61832B1C45BF070072450B /* AirCustom.cpp in Sources */,
</span><del>- 0FDE87F91DFD0C760064C390 /* CellContainer.cpp in Sources */,
</del><span class="cx"> 0F13912B16771C3A009CCB07 /* ProfilerProfiledBytecodes.cpp in Sources */,
</span><span class="cx"> 0FD3E40D1B618B6600C80E1E /* PropertyCondition.cpp in Sources */,
</span><span class="cx"> E35CA1531DBC3A5C00F83516 /* DOMJITHeapRange.cpp in Sources */,
</span></span></pre></div>
<a id="trunkSourceJavaScriptCoreheapCellContainercpp"></a>
<div class="delfile"><h4>Deleted: trunk/Source/JavaScriptCore/heap/CellContainer.cpp (209794 => 209795)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/JavaScriptCore/heap/CellContainer.cpp 2016-12-14 08:23:23 UTC (rev 209794)
+++ trunk/Source/JavaScriptCore/heap/CellContainer.cpp 2016-12-14 09:35:18 UTC (rev 209795)
</span><span class="lines">@@ -1,43 +0,0 @@
</span><del>-/*
- * Copyright (C) 2016 Apple Inc. All rights reserved.
- *
- * 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 "CellContainer.h"
-
-#include "MarkedBlockInlines.h"
-
-namespace JSC {
-
-bool CellContainer::isNewlyAllocated(HeapCell* cell) const
-{
- if (isLargeAllocation())
- return largeAllocation().isNewlyAllocated();
- MarkedBlock::Handle& handle = markedBlock().handle();
- return !handle.isNewlyAllocatedStale()
- && handle.isNewlyAllocated(cell);
-}
-
-} // namespace JSC
-
</del></span></pre></div>
<a id="trunkSourceJavaScriptCoreheapCellContainerh"></a>
<div class="modfile"><h4>Modified: trunk/Source/JavaScriptCore/heap/CellContainer.h (209794 => 209795)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/JavaScriptCore/heap/CellContainer.h 2016-12-14 08:23:23 UTC (rev 209794)
+++ trunk/Source/JavaScriptCore/heap/CellContainer.h 2016-12-14 09:35:18 UTC (rev 209795)
</span><span class="lines">@@ -84,8 +84,6 @@
</span><span class="cx"> bool isMarked(HeapCell*) const;
</span><span class="cx"> bool isMarked(HeapVersion markingVersion, HeapCell*) const;
</span><span class="cx">
</span><del>- bool isNewlyAllocated(HeapCell*) const;
-
</del><span class="cx"> void noteMarked();
</span><span class="cx"> void assertValidCell(VM&, HeapCell*) const;
</span><span class="cx">
</span></span></pre></div>
<a id="trunkSourceJavaScriptCoreheapMarkedAllocatorcpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/JavaScriptCore/heap/MarkedAllocator.cpp (209794 => 209795)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/JavaScriptCore/heap/MarkedAllocator.cpp 2016-12-14 08:23:23 UTC (rev 209794)
+++ trunk/Source/JavaScriptCore/heap/MarkedAllocator.cpp 2016-12-14 09:35:18 UTC (rev 209795)
</span><span class="lines">@@ -270,7 +270,6 @@
</span><span class="cx"> m_blocks.append(block);
</span><span class="cx"> if (m_blocks.capacity() != oldCapacity) {
</span><span class="cx"> forEachBitVector(
</span><del>- NoLockingNecessary,
</del><span class="cx"> [&] (FastBitVector& vector) {
</span><span class="cx"> ASSERT_UNUSED(vector, vector.numBits() == oldCapacity);
</span><span class="cx"> });
</span><span class="lines">@@ -279,7 +278,6 @@
</span><span class="cx">
</span><span class="cx"> LockHolder locker(m_bitvectorLock);
</span><span class="cx"> forEachBitVector(
</span><del>- locker,
</del><span class="cx"> [&] (FastBitVector& vector) {
</span><span class="cx"> vector.resize(m_blocks.capacity());
</span><span class="cx"> });
</span><span class="lines">@@ -291,7 +289,6 @@
</span><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> forEachBitVector(
</span><del>- NoLockingNecessary,
</del><span class="cx"> [&] (FastBitVector& vector) {
</span><span class="cx"> ASSERT_UNUSED(vector, !vector[index]);
</span><span class="cx"> });
</span><span class="lines">@@ -312,7 +309,6 @@
</span><span class="cx"> m_freeBlockIndices.append(block->index());
</span><span class="cx">
</span><span class="cx"> forEachBitVector(
</span><del>- holdLock(m_bitvectorLock),
</del><span class="cx"> [&] (FastBitVector& vector) {
</span><span class="cx"> vector[block->index()] = false;
</span><span class="cx"> });
</span><span class="lines">@@ -471,7 +467,6 @@
</span><span class="cx"> {
</span><span class="cx"> unsigned maxNameLength = 0;
</span><span class="cx"> forEachBitVectorWithName(
</span><del>- NoLockingNecessary,
</del><span class="cx"> [&] (FastBitVector&, const char* name) {
</span><span class="cx"> unsigned length = strlen(name);
</span><span class="cx"> maxNameLength = std::max(maxNameLength, length);
</span><span class="lines">@@ -478,7 +473,6 @@
</span><span class="cx"> });
</span><span class="cx">
</span><span class="cx"> forEachBitVectorWithName(
</span><del>- NoLockingNecessary,
</del><span class="cx"> [&] (FastBitVector& vector, const char* name) {
</span><span class="cx"> out.print(" ", name, ": ");
</span><span class="cx"> for (unsigned i = maxNameLength - strlen(name); i--;)
</span></span></pre></div>
<a id="trunkSourceJavaScriptCoreheapMarkedAllocatorh"></a>
<div class="modfile"><h4>Modified: trunk/Source/JavaScriptCore/heap/MarkedAllocator.h (209794 => 209795)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/JavaScriptCore/heap/MarkedAllocator.h 2016-12-14 08:23:23 UTC (rev 209794)
+++ trunk/Source/JavaScriptCore/heap/MarkedAllocator.h 2016-12-14 09:35:18 UTC (rev 209795)
</span><span class="lines">@@ -182,7 +182,7 @@
</span><span class="cx"> #undef MARKED_ALLOCATOR_BIT_ACCESSORS
</span><span class="cx">
</span><span class="cx"> template<typename Func>
</span><del>- void forEachBitVector(const AbstractLocker&, const Func& func)
</del><ins>+ void forEachBitVector(const Func& func)
</ins><span class="cx"> {
</span><span class="cx"> #define MARKED_ALLOCATOR_BIT_CALLBACK(lowerBitName, capitalBitName) \
</span><span class="cx"> func(m_ ## lowerBitName);
</span><span class="lines">@@ -191,7 +191,7 @@
</span><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> template<typename Func>
</span><del>- void forEachBitVectorWithName(const AbstractLocker&, const Func& func)
</del><ins>+ void forEachBitVectorWithName(const Func& func)
</ins><span class="cx"> {
</span><span class="cx"> #define MARKED_ALLOCATOR_BIT_CALLBACK(lowerBitName, capitalBitName) \
</span><span class="cx"> func(m_ ## lowerBitName, #capitalBitName);
</span></span></pre></div>
<a id="trunkSourceJavaScriptCoreheapMarkedBlockcpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/JavaScriptCore/heap/MarkedBlock.cpp (209794 => 209795)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/JavaScriptCore/heap/MarkedBlock.cpp 2016-12-14 08:23:23 UTC (rev 209794)
+++ trunk/Source/JavaScriptCore/heap/MarkedBlock.cpp 2016-12-14 09:35:18 UTC (rev 209795)
</span><span class="lines">@@ -44,7 +44,7 @@
</span><span class="cx"> if (computeBalance) {
</span><span class="cx"> balance++;
</span><span class="cx"> if (!(balance % 10))
</span><del>- dataLog("MarkedBlock Balance: ", balance, "\n");
</del><ins>+ dataLog(toCString("MarkedBlock Balance: ", balance, "\n"));
</ins><span class="cx"> }
</span><span class="cx"> void* blockSpace = tryFastAlignedMalloc(blockSize, blockSize);
</span><span class="cx"> if (!blockSpace)
</span><span class="lines">@@ -71,7 +71,7 @@
</span><span class="cx"> if (computeBalance) {
</span><span class="cx"> balance--;
</span><span class="cx"> if (!(balance % 10))
</span><del>- dataLog("MarkedBlock Balance: ", balance, "\n");
</del><ins>+ dataLog(toCString("MarkedBlock Balance: ", balance, "\n"));
</ins><span class="cx"> }
</span><span class="cx"> removeFromAllocator();
</span><span class="cx"> m_block->~MarkedBlock();
</span><span class="lines">@@ -85,7 +85,7 @@
</span><span class="cx"> , m_vm(&vm)
</span><span class="cx"> {
</span><span class="cx"> if (false)
</span><del>- dataLog(RawPointer(this), ": Allocated.\n");
</del><ins>+ dataLog(toCString(RawPointer(this), ": Allocated.\n"));
</ins><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> template<MarkedBlock::Handle::EmptyMode emptyMode, MarkedBlock::Handle::SweepMode sweepMode, MarkedBlock::Handle::SweepDestructionMode destructionMode, MarkedBlock::Handle::ScribbleMode scribbleMode, MarkedBlock::Handle::NewlyAllocatedMode newlyAllocatedMode, MarkedBlock::Handle::MarksMode marksMode>
</span><span class="lines">@@ -98,7 +98,7 @@
</span><span class="cx"> MarkedBlock& block = this->block();
</span><span class="cx">
</span><span class="cx"> if (false)
</span><del>- dataLog(RawPointer(this), "/", RawPointer(&block), ": MarkedBlock::Handle::specializedSweep!\n");
</del><ins>+ dataLog(toCString(RawPointer(this), "/", RawPointer(&block), ": MarkedBlock::Handle::specializedSweep!\n"));
</ins><span class="cx">
</span><span class="cx"> if (Options::useBumpAllocator()
</span><span class="cx"> && emptyMode == IsEmpty
</span><span class="lines">@@ -106,14 +106,11 @@
</span><span class="cx">
</span><span class="cx"> // This is an incredibly powerful assertion that checks the sanity of our block bits.
</span><span class="cx"> if (marksMode == MarksNotStale && !block.m_marks.isEmpty()) {
</span><del>- WTF::dataFile().atomically(
- [&] (PrintStream& out) {
- out.print("Block ", RawPointer(&block), ": marks not empty!\n");
- out.print("Block lock is held: ", block.m_lock.isHeld(), "\n");
- out.print("Marking version of block: ", block.m_markingVersion, "\n");
- out.print("Marking version of heap: ", space()->markingVersion(), "\n");
- UNREACHABLE_FOR_PLATFORM();
- });
</del><ins>+ dataLog(toCString("Block ", RawPointer(&block), ": marks not empty!\n"));
+ dataLog(toCString("Block lock is held: ", block.m_lock.isHeld(), "\n"));
+ dataLog(toCString("Marking version of block: ", block.m_markingVersion, "\n"));
+ dataLog(toCString("Marking version of heap: ", space()->markingVersion(), "\n"));
+ RELEASE_ASSERT_NOT_REACHED();
</ins><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> char* startOfLastCell = static_cast<char*>(cellAlign(block.atoms() + m_endAtom - 1));
</span><span class="lines">@@ -130,7 +127,7 @@
</span><span class="cx"> block.m_lock.unlock();
</span><span class="cx"> FreeList result = FreeList::bump(payloadEnd, payloadEnd - payloadBegin);
</span><span class="cx"> if (false)
</span><del>- dataLog("Quickly swept block ", RawPointer(this), " with cell size ", cellSize(), " and attributes ", m_attributes, ": ", result, "\n");
</del><ins>+ dataLog(toCString("Quickly swept block ", RawPointer(this), " with cell size ", cellSize(), " and attributes ", m_attributes, ": ", result, "\n"));
</ins><span class="cx"> return result;
</span><span class="cx"> }
</span><span class="cx">
</span><span class="lines">@@ -189,7 +186,7 @@
</span><span class="cx"> else if (isEmpty)
</span><span class="cx"> m_allocator->setIsEmpty(NoLockingNecessary, this, true);
</span><span class="cx"> if (false)
</span><del>- dataLog("Slowly swept block ", RawPointer(&block), " with cell size ", cellSize(), " and attributes ", m_attributes, ": ", result, "\n");
</del><ins>+ dataLog(toCString("Slowly swept block ", RawPointer(&block), " with cell size ", cellSize(), " and attributes ", m_attributes, ": ", result, "\n"));
</ins><span class="cx"> return result;
</span><span class="cx"> }
</span><span class="cx">
</span><span class="lines">@@ -265,7 +262,7 @@
</span><span class="cx"> HeapVersion markingVersion = space()->markingVersion();
</span><span class="cx"> bool marksAreUseful = !block().areMarksStale(markingVersion);
</span><span class="cx">
</span><del>- if (space()->isMarking())
</del><ins>+ if (space()->isMarking() && heap()->collectionScope() == CollectionScope::Full)
</ins><span class="cx"> marksAreUseful |= block().marksConveyLivenessDuringMarking(markingVersion);
</span><span class="cx">
</span><span class="cx"> if (!marksAreUseful)
</span><span class="lines">@@ -290,12 +287,12 @@
</span><span class="cx"> auto locker = holdLock(block().m_lock);
</span><span class="cx">
</span><span class="cx"> if (false)
</span><del>- dataLog(RawPointer(this), ": MarkedBlock::Handle::stopAllocating!\n");
</del><ins>+ dataLog(toCString(RawPointer(this), ": MarkedBlock::Handle::stopAllocating!\n"));
</ins><span class="cx"> ASSERT(!allocator()->isAllocated(NoLockingNecessary, this));
</span><span class="cx">
</span><span class="cx"> if (!isFreeListed()) {
</span><span class="cx"> if (false)
</span><del>- dataLog("There ain't no newly allocated.\n");
</del><ins>+ dataLog(toCString("There ain't no newly allocated.\n"));
</ins><span class="cx"> // This means that we either didn't use this block at all for allocation since last GC,
</span><span class="cx"> // or someone had already done stopAllocating() before.
</span><span class="cx"> ASSERT(freeList.allocationWillFail());
</span><span class="lines">@@ -303,7 +300,7 @@
</span><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> if (false)
</span><del>- dataLog("Free list: ", freeList, "\n");
</del><ins>+ dataLog(toCString("Free list: ", freeList, "\n"));
</ins><span class="cx">
</span><span class="cx"> // Roll back to a coherent state for Heap introspection. Cells newly
</span><span class="cx"> // allocated from our free list are not currently marked, so we need another
</span><span class="lines">@@ -322,7 +319,7 @@
</span><span class="cx"> freeList,
</span><span class="cx"> [&] (HeapCell* cell) {
</span><span class="cx"> if (false)
</span><del>- dataLog("Free cell: ", RawPointer(cell), "\n");
</del><ins>+ dataLog(toCString("Free cell: ", RawPointer(cell), "\n"));
</ins><span class="cx"> if (m_attributes.destruction == NeedsDestruction)
</span><span class="cx"> cell->zap();
</span><span class="cx"> clearNewlyAllocated(cell);
</span><span class="lines">@@ -349,13 +346,13 @@
</span><span class="cx"> auto locker = holdLock(block().m_lock);
</span><span class="cx">
</span><span class="cx"> if (false)
</span><del>- dataLog(RawPointer(this), ": MarkedBlock::Handle::resumeAllocating!\n");
</del><ins>+ dataLog(toCString(RawPointer(this), ": MarkedBlock::Handle::resumeAllocating!\n"));
</ins><span class="cx"> ASSERT(!allocator()->isAllocated(NoLockingNecessary, this));
</span><span class="cx"> ASSERT(!isFreeListed());
</span><span class="cx">
</span><span class="cx"> if (!hasAnyNewlyAllocated()) {
</span><span class="cx"> if (false)
</span><del>- dataLog("There ain't no newly allocated.\n");
</del><ins>+ dataLog(toCString("There ain't no newly allocated.\n"));
</ins><span class="cx"> // This means we had already exhausted the block when we stopped allocation.
</span><span class="cx"> return FreeList();
</span><span class="cx"> }
</span><span class="lines">@@ -404,7 +401,7 @@
</span><span class="cx"> if (handle().allocator()->isAllocated(holdLock(allocator->bitvectorLock()), &handle())
</span><span class="cx"> || !marksConveyLivenessDuringMarking(markingVersion)) {
</span><span class="cx"> if (false)
</span><del>- dataLog(RawPointer(this), ": Clearing marks without doing anything else.\n");
</del><ins>+ dataLog(toCString(RawPointer(this), ": Clearing marks without doing anything else.\n"));
</ins><span class="cx"> // We already know that the block is full and is already recognized as such, or that the
</span><span class="cx"> // block did not survive the previous GC. So, we can clear mark bits the old fashioned
</span><span class="cx"> // way. Note that it's possible for such a block to have newlyAllocated with an up-to-
</span><span class="lines">@@ -414,7 +411,7 @@
</span><span class="cx"> m_marks.clearAll();
</span><span class="cx"> } else {
</span><span class="cx"> if (false)
</span><del>- dataLog(RawPointer(this), ": Doing things.\n");
</del><ins>+ dataLog(toCString(RawPointer(this), ": Doing things.\n"));
</ins><span class="cx"> HeapVersion newlyAllocatedVersion = space()->newlyAllocatedVersion();
</span><span class="cx"> if (handle().m_newlyAllocatedVersion == newlyAllocatedVersion) {
</span><span class="cx"> // Merge the contents of marked into newlyAllocated. If we get the full set of bits
</span><span class="lines">@@ -480,7 +477,7 @@
</span><span class="cx"> {
</span><span class="cx"> auto locker = holdLock(block().m_lock);
</span><span class="cx"> if (false)
</span><del>- dataLog(RawPointer(this), ": MarkedBlock::Handle::didConsumeFreeList!\n");
</del><ins>+ dataLog(toCString(RawPointer(this), ": MarkedBlock::Handle::didConsumeFreeList!\n"));
</ins><span class="cx"> ASSERT(isFreeListed());
</span><span class="cx"> m_isFreeListed = false;
</span><span class="cx"> allocator()->setIsAllocated(NoLockingNecessary, this, true);
</span><span class="lines">@@ -576,16 +573,6 @@
</span><span class="cx"> }
</span><span class="cx"> #endif
</span><span class="cx">
</span><del>-void MarkedBlock::Handle::dumpState(PrintStream& out)
-{
- CommaPrinter comma;
- allocator()->forEachBitVectorWithName(
- holdLock(allocator()->bitvectorLock()),
- [&] (FastBitVector& bitvector, const char* name) {
- out.print(comma, name, ":", bitvector[index()] ? "YES" : "no");
- });
-}
-
</del><span class="cx"> } // namespace JSC
</span><span class="cx">
</span><span class="cx"> namespace WTF {
</span></span></pre></div>
<a id="trunkSourceJavaScriptCoreheapMarkedBlockh"></a>
<div class="modfile"><h4>Modified: trunk/Source/JavaScriptCore/heap/MarkedBlock.h (209794 => 209795)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/JavaScriptCore/heap/MarkedBlock.h 2016-12-14 08:23:23 UTC (rev 209794)
+++ trunk/Source/JavaScriptCore/heap/MarkedBlock.h 2016-12-14 09:35:18 UTC (rev 209795)
</span><span class="lines">@@ -187,8 +187,6 @@
</span><span class="cx"> void didAddToAllocator(MarkedAllocator*, size_t index);
</span><span class="cx"> void didRemoveFromAllocator();
</span><span class="cx">
</span><del>- void dumpState(PrintStream&);
-
</del><span class="cx"> private:
</span><span class="cx"> Handle(Heap&, void*);
</span><span class="cx">
</span><span class="lines">@@ -300,9 +298,6 @@
</span><span class="cx">
</span><span class="cx"> void resetMarks();
</span><span class="cx">
</span><del>- bool isMarkedRaw(const void* p);
- HeapVersion markingVersion() const { return m_markingVersion; }
-
</del><span class="cx"> private:
</span><span class="cx"> static const size_t atomAlignmentMask = atomSize - 1;
</span><span class="cx">
</span><span class="lines">@@ -523,14 +518,9 @@
</span><span class="cx"> block().assertMarksNotStale();
</span><span class="cx"> }
</span><span class="cx">
</span><del>-inline bool MarkedBlock::isMarkedRaw(const void* p)
-{
- return m_marks.get(atomNumber(p));
-}
-
</del><span class="cx"> inline bool MarkedBlock::isMarked(HeapVersion markingVersion, const void* p)
</span><span class="cx"> {
</span><del>- return areMarksStale(markingVersion) ? false : isMarkedRaw(p);
</del><ins>+ return areMarksStale(markingVersion) ? false : m_marks.get(atomNumber(p));
</ins><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> inline bool MarkedBlock::isMarkedConcurrently(HeapVersion markingVersion, const void* p)
</span></span></pre></div>
<a id="trunkSourceJavaScriptCoreheapMarkedBlockInlinesh"></a>
<div class="modfile"><h4>Modified: trunk/Source/JavaScriptCore/heap/MarkedBlockInlines.h (209794 => 209795)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/JavaScriptCore/heap/MarkedBlockInlines.h 2016-12-14 08:23:23 UTC (rev 209794)
+++ trunk/Source/JavaScriptCore/heap/MarkedBlockInlines.h 2016-12-14 09:35:18 UTC (rev 209795)
</span><span class="lines">@@ -28,7 +28,6 @@
</span><span class="cx"> #include "MarkedAllocator.h"
</span><span class="cx"> #include "MarkedBlock.h"
</span><span class="cx"> #include "MarkedSpace.h"
</span><del>-#include "VM.h"
</del><span class="cx">
</span><span class="cx"> namespace JSC {
</span><span class="cx">
</span><span class="lines">@@ -52,8 +51,6 @@
</span><span class="cx"> // back" state only makes sense when we're in a concurrent collection and have to be
</span><span class="cx"> // conservative.
</span><span class="cx"> ASSERT(space()->isMarking());
</span><del>- if (heap()->collectionScope() != CollectionScope::Full)
- return false;
</del><span class="cx"> return m_markingVersion == MarkedSpace::nullVersion
</span><span class="cx"> || MarkedSpace::nextVersion(m_markingVersion) == markingVersion;
</span><span class="cx"> }
</span></span></pre></div>
<a id="trunkSourceJavaScriptCoreheapSlotVisitorcpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/JavaScriptCore/heap/SlotVisitor.cpp (209794 => 209795)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/JavaScriptCore/heap/SlotVisitor.cpp 2016-12-14 08:23:23 UTC (rev 209794)
+++ trunk/Source/JavaScriptCore/heap/SlotVisitor.cpp 2016-12-14 09:35:18 UTC (rev 209795)
</span><span class="lines">@@ -135,75 +135,18 @@
</span><span class="cx">
</span><span class="cx"> ASSERT(!m_isCheckingForDefaultMarkViolation);
</span><span class="cx">
</span><del>- auto validateCell = [&] (JSCell* jsCell) {
- StructureID structureID = jsCell->structureID();
-
- auto die = [&] (const char* text) {
- WTF::dataFile().atomically(
- [&] (PrintStream& out) {
- out.print(text);
- out.print("GC type: ", heap()->collectionScope(), "\n");
- out.print("Object at: ", RawPointer(jsCell), "\n");
-#if USE(JSVALUE64)
- out.print("Structure ID: ", structureID, " (0x", format("%x", structureID), ")\n");
- out.print("Structure ID table size: ", heap()->structureIDTable().size(), "\n");
-#else
- out.print("Structure: ", RawPointer(structureID), "\n");
-#endif
- out.print("Object contents:");
- for (unsigned i = 0; i < 2; ++i)
- out.print(" ", format("0x%016llx", bitwise_cast<uint64_t*>(jsCell)[i]));
- out.print("\n");
- CellContainer container = jsCell->cellContainer();
- out.print("Is marked: ", container.isMarked(jsCell), "\n");
- out.print("Is newly allocated: ", container.isNewlyAllocated(jsCell), "\n");
- if (container.isMarkedBlock()) {
- MarkedBlock& block = container.markedBlock();
- out.print("Block: ", RawPointer(&block), "\n");
- block.handle().dumpState(out);
- out.print("\n");
- out.print("Is marked raw: ", block.isMarkedRaw(jsCell), "\n");
- out.print("Marking version: ", block.markingVersion(), "\n");
- out.print("Heap marking version: ", heap()->objectSpace().markingVersion(), "\n");
- out.print("Is newly allocated raw: ", block.handle().isNewlyAllocated(jsCell), "\n");
- out.print("Newly allocated version: ", block.handle().newlyAllocatedVersion(), "\n");
- out.print("Heap newly allocated version: ", heap()->objectSpace().newlyAllocatedVersion(), "\n");
- }
- UNREACHABLE_FOR_PLATFORM();
- });
- };
-
- // It's not OK for the structure to be null at any GC scan point. We must not GC while
- // an object is not fully initialized.
- if (!structureID)
- die("GC scan found corrupt object: structureID is zero!\n");
-
- // It's not OK for the structure to be nuked at any GC scan point.
- if (isNuked(structureID))
- die("GC scan found object in bad state: structureID is nuked!\n");
-
-#if USE(JSVALUE64)
- // This detects the worst of the badness.
- if (structureID >= heap()->structureIDTable().size())
- die("GC scan found corrupt object: structureID is out of bounds!\n");
-#endif
- };
-
- // In debug mode, we validate before marking since this makes it clearer what the problem
- // was. It's also slower, so we don't do it normally.
- if (!ASSERT_DISABLED && heapCell->cellKind() == HeapCell::JSCell)
- validateCell(static_cast<JSCell*>(heapCell));
-
</del><span class="cx"> if (Heap::testAndSetMarked(m_markingVersion, heapCell))
</span><span class="cx"> return;
</span><span class="cx">
</span><span class="cx"> switch (heapCell->cellKind()) {
</span><span class="cx"> case HeapCell::JSCell: {
</span><del>- // We have ample budget to perform validation here.
-
</del><span class="cx"> JSCell* jsCell = static_cast<JSCell*>(heapCell);
</span><del>- validateCell(jsCell);
</del><span class="cx">
</span><ins>+ if (!jsCell->structure()) {
+ ASSERT_NOT_REACHED();
+ return;
+ }
+
</ins><span class="cx"> jsCell->setCellState(CellState::Grey);
</span><span class="cx">
</span><span class="cx"> appendToMarkStack(jsCell);
</span></span></pre></div>
<a id="trunkSourceJavaScriptCoreruntimeOptionsh"></a>
<div class="modfile"><h4>Modified: trunk/Source/JavaScriptCore/runtime/Options.h (209794 => 209795)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/JavaScriptCore/runtime/Options.h 2016-12-14 08:23:23 UTC (rev 209794)
+++ trunk/Source/JavaScriptCore/runtime/Options.h 2016-12-14 09:35:18 UTC (rev 209795)
</span><span class="lines">@@ -184,7 +184,7 @@
</span><span class="cx"> v(bool, verboseSanitizeStack, false, Normal, nullptr) \
</span><span class="cx"> v(bool, useGenerationalGC, true, Normal, nullptr) \
</span><span class="cx"> v(bool, useConcurrentBarriers, true, Normal, nullptr) \
</span><del>- v(bool, useConcurrentGC, true, Normal, nullptr) \
</del><ins>+ v(bool, useConcurrentGC, false, Normal, nullptr) \
</ins><span class="cx"> v(bool, collectContinuously, false, Normal, nullptr) \
</span><span class="cx"> v(double, collectContinuouslyPeriodMS, 1, Normal, nullptr) \
</span><span class="cx"> v(bool, forceFencedBarrier, false, Normal, nullptr) \
</span></span></pre></div>
<a id="trunkSourceJavaScriptCoreruntimeStructureIDTableh"></a>
<div class="modfile"><h4>Modified: trunk/Source/JavaScriptCore/runtime/StructureIDTable.h (209794 => 209795)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/JavaScriptCore/runtime/StructureIDTable.h 2016-12-14 08:23:23 UTC (rev 209794)
+++ trunk/Source/JavaScriptCore/runtime/StructureIDTable.h 2016-12-14 09:35:18 UTC (rev 209795)
</span><span class="lines">@@ -90,8 +90,6 @@
</span><span class="cx"> StructureID allocateID(Structure*);
</span><span class="cx">
</span><span class="cx"> void flushOldTables();
</span><del>-
- size_t size() const { return m_size; }
</del><span class="cx">
</span><span class="cx"> private:
</span><span class="cx"> void resize(size_t newCapacity);
</span><span class="lines">@@ -123,9 +121,7 @@
</span><span class="cx"> inline Structure* StructureIDTable::get(StructureID structureID)
</span><span class="cx"> {
</span><span class="cx"> #if USE(JSVALUE64)
</span><del>- ASSERT_WITH_SECURITY_IMPLICATION(structureID);
- ASSERT_WITH_SECURITY_IMPLICATION(!isNuked(structureID));
- ASSERT_WITH_SECURITY_IMPLICATION(structureID < m_capacity);
</del><ins>+ ASSERT_WITH_SECURITY_IMPLICATION(structureID && structureID < m_capacity);
</ins><span class="cx"> return table()[structureID].structure;
</span><span class="cx"> #else
</span><span class="cx"> return structureID;
</span></span></pre></div>
<a id="trunkSourceWTFChangeLog"></a>
<div class="modfile"><h4>Modified: trunk/Source/WTF/ChangeLog (209794 => 209795)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WTF/ChangeLog 2016-12-14 08:23:23 UTC (rev 209794)
+++ trunk/Source/WTF/ChangeLog 2016-12-14 09:35:18 UTC (rev 209795)
</span><span class="lines">@@ -1,3 +1,37 @@
</span><ins>+2016-12-14 Gavin Barraclough <barraclough@apple.com>
+
+ MarkedBlock::marksConveyLivenessDuringMarking should take into account collection scope
+ https://bugs.webkit.org/show_bug.cgi?id=165741
+
+ Unreviewed rollout due to performance regression.
+
+ * WTF.xcodeproj/project.pbxproj:
+ * wtf/CMakeLists.txt:
+ * wtf/DataLog.cpp:
+ (WTF::initializeLogFileOnce):
+ (WTF::initializeLogFile):
+ (WTF::dataFile):
+ * wtf/DataLog.h:
+ * wtf/LockedPrintStream.cpp: Removed.
+ * wtf/LockedPrintStream.h: Removed.
+ * wtf/PrintStream.cpp:
+ (WTF::PrintStream::printfVariableFormat): Deleted.
+ (WTF::PrintStream::begin): Deleted.
+ (WTF::PrintStream::end): Deleted.
+ * wtf/PrintStream.h:
+ (WTF::PrintStream::print):
+ (WTF::PrintStream::println):
+ (WTF::PrintStream::atomically): Deleted.
+ (WTF::PrintStream::printImpl): Deleted.
+ (): Deleted.
+ (WTF::>::unpack): Deleted.
+ (WTF::FormatImpl::FormatImpl): Deleted.
+ (WTF::FormatImpl::dump): Deleted.
+ (WTF::format): Deleted.
+ * wtf/RecursiveLockAdapter.h: Removed.
+ * wtf/WordLock.cpp:
+ * wtf/WordLock.h:
+
</ins><span class="cx"> 2016-12-13 JF Bastien <jfbastien@apple.com>
</span><span class="cx">
</span><span class="cx"> std::expected: fix rvalue forwarding issues
</span></span></pre></div>
<a id="trunkSourceWTFWTFxcodeprojprojectpbxproj"></a>
<div class="modfile"><h4>Modified: trunk/Source/WTF/WTF.xcodeproj/project.pbxproj (209794 => 209795)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WTF/WTF.xcodeproj/project.pbxproj 2016-12-14 08:23:23 UTC (rev 209794)
+++ trunk/Source/WTF/WTF.xcodeproj/project.pbxproj 2016-12-14 09:35:18 UTC (rev 209795)
</span><span class="lines">@@ -30,8 +30,6 @@
</span><span class="cx"> 0F43D8F21DB5ADDC00108FB6 /* AutomaticThread.h in Headers */ = {isa = PBXBuildFile; fileRef = 0F43D8F01DB5ADDC00108FB6 /* AutomaticThread.h */; };
</span><span class="cx"> 0F4570431BE5B58F0062A629 /* Dominators.h in Headers */ = {isa = PBXBuildFile; fileRef = 0F4570421BE5B58F0062A629 /* Dominators.h */; };
</span><span class="cx"> 0F4570451BE834410062A629 /* BubbleSort.h in Headers */ = {isa = PBXBuildFile; fileRef = 0F4570441BE834410062A629 /* BubbleSort.h */; };
</span><del>- 0F60F32F1DFCBD1B00416D6C /* LockedPrintStream.cpp in Sources */ = {isa = PBXBuildFile; fileRef = 0F60F32D1DFCBD1B00416D6C /* LockedPrintStream.cpp */; };
- 0F60F3301DFCBD1B00416D6C /* LockedPrintStream.h in Headers */ = {isa = PBXBuildFile; fileRef = 0F60F32E1DFCBD1B00416D6C /* LockedPrintStream.h */; };
</del><span class="cx"> 0F66B28A1DC97BAB004A1D3F /* ClockType.cpp in Sources */ = {isa = PBXBuildFile; fileRef = 0F66B2801DC97BAB004A1D3F /* ClockType.cpp */; };
</span><span class="cx"> 0F66B28B1DC97BAB004A1D3F /* ClockType.h in Headers */ = {isa = PBXBuildFile; fileRef = 0F66B2811DC97BAB004A1D3F /* ClockType.h */; };
</span><span class="cx"> 0F66B28C1DC97BAB004A1D3F /* MonotonicTime.cpp in Sources */ = {isa = PBXBuildFile; fileRef = 0F66B2821DC97BAB004A1D3F /* MonotonicTime.cpp */; };
</span><span class="lines">@@ -65,7 +63,6 @@
</span><span class="cx"> 0FDB698E1B7C643A000C1078 /* Condition.h in Headers */ = {isa = PBXBuildFile; fileRef = 0FDB698D1B7C643A000C1078 /* Condition.h */; };
</span><span class="cx"> 0FDDBFA71666DFA300C55FEF /* StringPrintStream.cpp in Sources */ = {isa = PBXBuildFile; fileRef = 0FDDBFA51666DFA300C55FEF /* StringPrintStream.cpp */; };
</span><span class="cx"> 0FDDBFA81666DFA300C55FEF /* StringPrintStream.h in Headers */ = {isa = PBXBuildFile; fileRef = 0FDDBFA61666DFA300C55FEF /* StringPrintStream.h */; };
</span><del>- 0FDE87F71DFD07CC0064C390 /* RecursiveLockAdapter.h in Headers */ = {isa = PBXBuildFile; fileRef = 0FDE87F61DFD07CC0064C390 /* RecursiveLockAdapter.h */; };
</del><span class="cx"> 0FE1646A1B6FFC9600400E7C /* Lock.cpp in Sources */ = {isa = PBXBuildFile; fileRef = 0FE164681B6FFC9600400E7C /* Lock.cpp */; };
</span><span class="cx"> 0FE1646B1B6FFC9600400E7C /* Lock.h in Headers */ = {isa = PBXBuildFile; fileRef = 0FE164691B6FFC9600400E7C /* Lock.h */; };
</span><span class="cx"> 0FE4479C1B7AAA03009498EB /* WordLock.cpp in Sources */ = {isa = PBXBuildFile; fileRef = 0FE4479A1B7AAA03009498EB /* WordLock.cpp */; };
</span><span class="lines">@@ -391,8 +388,6 @@
</span><span class="cx"> 0F43D8F01DB5ADDC00108FB6 /* AutomaticThread.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = AutomaticThread.h; sourceTree = "<group>"; };
</span><span class="cx"> 0F4570421BE5B58F0062A629 /* Dominators.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = Dominators.h; sourceTree = "<group>"; };
</span><span class="cx"> 0F4570441BE834410062A629 /* BubbleSort.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = BubbleSort.h; sourceTree = "<group>"; };
</span><del>- 0F60F32D1DFCBD1B00416D6C /* LockedPrintStream.cpp */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp; path = LockedPrintStream.cpp; sourceTree = "<group>"; };
- 0F60F32E1DFCBD1B00416D6C /* LockedPrintStream.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = LockedPrintStream.h; sourceTree = "<group>"; };
</del><span class="cx"> 0F66B2801DC97BAB004A1D3F /* ClockType.cpp */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp; path = ClockType.cpp; sourceTree = "<group>"; };
</span><span class="cx"> 0F66B2811DC97BAB004A1D3F /* ClockType.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = ClockType.h; sourceTree = "<group>"; };
</span><span class="cx"> 0F66B2821DC97BAB004A1D3F /* MonotonicTime.cpp */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp; path = MonotonicTime.cpp; sourceTree = "<group>"; };
</span><span class="lines">@@ -426,7 +421,6 @@
</span><span class="cx"> 0FDB698D1B7C643A000C1078 /* Condition.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = Condition.h; sourceTree = "<group>"; };
</span><span class="cx"> 0FDDBFA51666DFA300C55FEF /* StringPrintStream.cpp */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp; path = StringPrintStream.cpp; sourceTree = "<group>"; };
</span><span class="cx"> 0FDDBFA61666DFA300C55FEF /* StringPrintStream.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = StringPrintStream.h; sourceTree = "<group>"; };
</span><del>- 0FDE87F61DFD07CC0064C390 /* RecursiveLockAdapter.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = RecursiveLockAdapter.h; sourceTree = "<group>"; };
</del><span class="cx"> 0FE164681B6FFC9600400E7C /* Lock.cpp */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp; path = Lock.cpp; sourceTree = "<group>"; };
</span><span class="cx"> 0FE164691B6FFC9600400E7C /* Lock.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = Lock.h; sourceTree = "<group>"; };
</span><span class="cx"> 0FE4479A1B7AAA03009498EB /* WordLock.cpp */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp; path = WordLock.cpp; sourceTree = "<group>"; };
</span><span class="lines">@@ -864,6 +858,7 @@
</span><span class="cx"> A876DBD6151816E500DADB95 /* wtf */ = {
</span><span class="cx"> isa = PBXGroup;
</span><span class="cx"> children = (
</span><ins>+ AD7C434A1DD2A4A70026888B /* Expected.h */,
</ins><span class="cx"> 2CDED0F018115C3F004DBA70 /* cf */,
</span><span class="cx"> E4A0AD3B1A96251900536DF6 /* cocoa */,
</span><span class="cx"> A8A47281151A825A004123FF /* dtoa */,
</span><span class="lines">@@ -931,7 +926,6 @@
</span><span class="cx"> A8A47297151A825A004123FF /* dtoa.cpp */,
</span><span class="cx"> A8A47298151A825A004123FF /* dtoa.h */,
</span><span class="cx"> 1AEA88E11D6BBCF400E5AD64 /* EnumTraits.h */,
</span><del>- AD7C434A1DD2A4A70026888B /* Expected.h */,
</del><span class="cx"> A8A4729F151A825A004123FF /* ExportMacros.h */,
</span><span class="cx"> 0F7C5FB51D885CF20044F5E2 /* FastBitVector.cpp */,
</span><span class="cx"> 0FD81AC4154FB22E00983E72 /* FastBitVector.h */,
</span><span class="lines">@@ -972,8 +966,6 @@
</span><span class="cx"> 0FE164681B6FFC9600400E7C /* Lock.cpp */,
</span><span class="cx"> 0FE164691B6FFC9600400E7C /* Lock.h */,
</span><span class="cx"> 0F0FCDDD1DD167F900CCAB53 /* LockAlgorithm.h */,
</span><del>- 0F60F32D1DFCBD1B00416D6C /* LockedPrintStream.cpp */,
- 0F60F32E1DFCBD1B00416D6C /* LockedPrintStream.h */,
</del><span class="cx"> A8A472C3151A825A004123FF /* Locker.h */,
</span><span class="cx"> 513E170A1CD7D5BF00E3650B /* LoggingAccumulator.h */,
</span><span class="cx"> A8A472C6151A825A004123FF /* MainThread.cpp */,
</span><span class="lines">@@ -1033,7 +1025,6 @@
</span><span class="cx"> A8A472FD151A825B004123FF /* RandomNumberSeed.h */,
</span><span class="cx"> 0F725CAB1C50461600AD943A /* RangeSet.h */,
</span><span class="cx"> 0F87105916643F190090B0AD /* RawPointer.h */,
</span><del>- 0FDE87F61DFD07CC0064C390 /* RecursiveLockAdapter.h */,
</del><span class="cx"> A8A472FE151A825B004123FF /* RedBlackTree.h */,
</span><span class="cx"> 26299B6D17A9E5B800ADEBE5 /* Ref.h */,
</span><span class="cx"> A8A472FF151A825B004123FF /* RefCounted.h */,
</span><span class="lines">@@ -1395,7 +1386,6 @@
</span><span class="cx"> 7CDD7FFA186D2A54007433CD /* IteratorRange.h in Headers */,
</span><span class="cx"> 0F43D8F21DB5ADDC00108FB6 /* AutomaticThread.h in Headers */,
</span><span class="cx"> 93AC91A818942FC400244939 /* LChar.h in Headers */,
</span><del>- 0F60F3301DFCBD1B00416D6C /* LockedPrintStream.h in Headers */,
</del><span class="cx"> 539EB0631D55284200C82EF7 /* LEBDecoder.h in Headers */,
</span><span class="cx"> A70DA0851799F04D00529A9B /* ListDump.h in Headers */,
</span><span class="cx"> 0F66B2931DC97BAB004A1D3F /* WallTime.h in Headers */,
</span><span class="lines">@@ -1469,7 +1459,6 @@
</span><span class="cx"> 0FEB3DCF1BB5D684009D7AAD /* SharedTask.h in Headers */,
</span><span class="cx"> A8A47423151A825B004123FF /* SimpleStats.h in Headers */,
</span><span class="cx"> A8A47424151A825B004123FF /* SinglyLinkedList.h in Headers */,
</span><del>- 0FDE87F71DFD07CC0064C390 /* RecursiveLockAdapter.h in Headers */,
</del><span class="cx"> A748745317A0BDAE00FA04CB /* SixCharacterHash.h in Headers */,
</span><span class="cx"> 79EC70611C99F9BC003A3AE2 /* SmallPtrSet.h in Headers */,
</span><span class="cx"> A8A47426151A825B004123FF /* Spectrum.h in Headers */,
</span><span class="lines">@@ -1665,7 +1654,6 @@
</span><span class="cx"> A8A473B0151A825B004123FF /* double-conversion.cc in Sources */,
</span><span class="cx"> A8A473BA151A825B004123FF /* dtoa.cpp in Sources */,
</span><span class="cx"> A8A473B3151A825B004123FF /* fast-dtoa.cc in Sources */,
</span><del>- 0F60F32F1DFCBD1B00416D6C /* LockedPrintStream.cpp in Sources */,
</del><span class="cx"> A8A473C3151A825B004123FF /* FastMalloc.cpp in Sources */,
</span><span class="cx"> 0F9D3360165DBA73005AD387 /* FilePrintStream.cpp in Sources */,
</span><span class="cx"> A8A473B5151A825B004123FF /* fixed-dtoa.cc in Sources */,
</span></span></pre></div>
<a id="trunkSourceWTFwtfCMakeListstxt"></a>
<div class="modfile"><h4>Modified: trunk/Source/WTF/wtf/CMakeLists.txt (209794 => 209795)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WTF/wtf/CMakeLists.txt 2016-12-14 08:23:23 UTC (rev 209794)
+++ trunk/Source/WTF/wtf/CMakeLists.txt 2016-12-14 09:35:18 UTC (rev 209795)
</span><span class="lines">@@ -57,7 +57,6 @@
</span><span class="cx"> ListHashSet.h
</span><span class="cx"> Lock.h
</span><span class="cx"> LockAlgorithm.h
</span><del>- LockedPrintStream.h
</del><span class="cx"> Locker.h
</span><span class="cx"> MD5.h
</span><span class="cx"> MainThread.h
</span><span class="lines">@@ -93,7 +92,6 @@
</span><span class="cx"> RandomNumberSeed.h
</span><span class="cx"> RangeSet.h
</span><span class="cx"> RawPointer.h
</span><del>- RecursiveLockAdapter.h
</del><span class="cx"> RedBlackTree.h
</span><span class="cx"> Ref.h
</span><span class="cx"> RefCounted.h
</span><span class="lines">@@ -200,7 +198,6 @@
</span><span class="cx"> GregorianDateTime.cpp
</span><span class="cx"> HashTable.cpp
</span><span class="cx"> Lock.cpp
</span><del>- LockedPrintStream.cpp
</del><span class="cx"> MD5.cpp
</span><span class="cx"> MainThread.cpp
</span><span class="cx"> MediaTime.cpp
</span></span></pre></div>
<a id="trunkSourceWTFwtfDataLogcpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WTF/wtf/DataLog.cpp (209794 => 209795)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WTF/wtf/DataLog.cpp 2016-12-14 08:23:23 UTC (rev 209794)
+++ trunk/Source/WTF/wtf/DataLog.cpp 2016-12-14 09:35:18 UTC (rev 209795)
</span><span class="lines">@@ -28,10 +28,7 @@
</span><span class="cx"> #include <stdarg.h>
</span><span class="cx"> #include <string.h>
</span><span class="cx"> #include <wtf/FilePrintStream.h>
</span><del>-#include <wtf/LockedPrintStream.h>
</del><span class="cx"> #include <wtf/Threading.h>
</span><del>-#include <mutex>
-#include <thread>
</del><span class="cx">
</span><span class="cx"> #if OS(UNIX) || OS(DARWIN)
</span><span class="cx"> #include <unistd.h>
</span><span class="lines">@@ -50,14 +47,16 @@
</span><span class="cx">
</span><span class="cx"> namespace WTF {
</span><span class="cx">
</span><del>-static PrintStream* s_file;
</del><ins>+#if USE(PTHREADS)
+static pthread_once_t initializeLogFileOnceKey = PTHREAD_ONCE_INIT;
+#endif
</ins><span class="cx">
</span><del>-static uint64_t s_fileData[(sizeof(FilePrintStream) + 7) / 8];
</del><ins>+static FilePrintStream* file;
</ins><span class="cx">
</span><ins>+static uint64_t fileData[(sizeof(FilePrintStream) + 7) / 8];
+
</ins><span class="cx"> static void initializeLogFileOnce()
</span><span class="cx"> {
</span><del>- FilePrintStream* file = nullptr;
-
</del><span class="cx"> #if DATA_LOG_TO_FILE
</span><span class="cx"> const long maxPathLength = 1024;
</span><span class="cx">
</span><span class="lines">@@ -118,38 +117,29 @@
</span><span class="cx"> WTFLogAlways("Warning: Could not open DataLog file %s for writing.\n", actualFilename);
</span><span class="cx"> }
</span><span class="cx"> #endif // DATA_LOG_TO_FILE
</span><del>-
- bool wrapWithLocked = true;
-
</del><span class="cx"> if (!file) {
</span><span class="cx"> // Use placement new; this makes it easier to use dataLog() to debug
</span><span class="cx"> // fastMalloc.
</span><del>- file = new (s_fileData) FilePrintStream(stderr, FilePrintStream::Borrow);
- wrapWithLocked = false;
</del><ins>+ file = new (fileData) FilePrintStream(stderr, FilePrintStream::Borrow);
</ins><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> setvbuf(file->file(), 0, _IONBF, 0); // Prefer unbuffered output, so that we get a full log upon crash or deadlock.
</span><del>-
- if (wrapWithLocked)
- s_file = new LockedPrintStream(std::unique_ptr<FilePrintStream>(file));
- else
- s_file = file;
</del><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> static void initializeLogFile()
</span><span class="cx"> {
</span><del>- static std::once_flag once;
- std::call_once(
- once,
- [] {
- initializeLogFileOnce();
- });
</del><ins>+#if USE(PTHREADS)
+ pthread_once(&initializeLogFileOnceKey, initializeLogFileOnce);
+#else
+ if (!file)
+ initializeLogFileOnce();
+#endif
</ins><span class="cx"> }
</span><span class="cx">
</span><del>-PrintStream& dataFile()
</del><ins>+FilePrintStream& dataFile()
</ins><span class="cx"> {
</span><span class="cx"> initializeLogFile();
</span><del>- return *s_file;
</del><ins>+ return *file;
</ins><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> void dataLogFV(const char* format, va_list argList)
</span></span></pre></div>
<a id="trunkSourceWTFwtfDataLogh"></a>
<div class="modfile"><h4>Modified: trunk/Source/WTF/wtf/DataLog.h (209794 => 209795)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WTF/wtf/DataLog.h 2016-12-14 08:23:23 UTC (rev 209794)
+++ trunk/Source/WTF/wtf/DataLog.h 2016-12-14 09:35:18 UTC (rev 209795)
</span><span class="lines">@@ -28,12 +28,12 @@
</span><span class="cx">
</span><span class="cx"> #include <stdarg.h>
</span><span class="cx"> #include <stdio.h>
</span><del>-#include <wtf/PrintStream.h>
</del><ins>+#include <wtf/FilePrintStream.h>
</ins><span class="cx"> #include <wtf/StdLibExtras.h>
</span><span class="cx">
</span><span class="cx"> namespace WTF {
</span><span class="cx">
</span><del>-WTF_EXPORT_PRIVATE PrintStream& dataFile();
</del><ins>+WTF_EXPORT_PRIVATE FilePrintStream& dataFile();
</ins><span class="cx">
</span><span class="cx"> WTF_EXPORT_PRIVATE void dataLogFV(const char* format, va_list) WTF_ATTRIBUTE_PRINTF(1, 0);
</span><span class="cx"> WTF_EXPORT_PRIVATE void dataLogF(const char* format, ...) WTF_ATTRIBUTE_PRINTF(1, 2);
</span></span></pre></div>
<a id="trunkSourceWTFwtfLockedPrintStreamcpp"></a>
<div class="delfile"><h4>Deleted: trunk/Source/WTF/wtf/LockedPrintStream.cpp (209794 => 209795)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WTF/wtf/LockedPrintStream.cpp 2016-12-14 08:23:23 UTC (rev 209794)
+++ trunk/Source/WTF/wtf/LockedPrintStream.cpp 2016-12-14 09:35:18 UTC (rev 209795)
</span><span class="lines">@@ -1,64 +0,0 @@
</span><del>-/*
- * Copyright (C) 2016 Apple Inc. All rights reserved.
- *
- * 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 "LockedPrintStream.h"
-
-namespace WTF {
-
-LockedPrintStream::LockedPrintStream(std::unique_ptr<PrintStream> target)
- : m_target(WTFMove(target))
-{
-}
-
-LockedPrintStream::~LockedPrintStream()
-{
-}
-
-void LockedPrintStream::vprintf(const char* format, va_list args)
-{
- auto locker = holdLock(m_lock);
- m_target->vprintf(format, args);
-}
-
-void LockedPrintStream::flush()
-{
- auto locker = holdLock(m_lock);
- m_target->flush();
-}
-
-PrintStream& LockedPrintStream::begin()
-{
- m_lock.lock();
- return *m_target;
-}
-
-void LockedPrintStream::end()
-{
- m_lock.unlock();
-}
-
-} // namespace WTF
-
</del></span></pre></div>
<a id="trunkSourceWTFwtfLockedPrintStreamh"></a>
<div class="delfile"><h4>Deleted: trunk/Source/WTF/wtf/LockedPrintStream.h (209794 => 209795)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WTF/wtf/LockedPrintStream.h 2016-12-14 08:23:23 UTC (rev 209794)
+++ trunk/Source/WTF/wtf/LockedPrintStream.h 2016-12-14 09:35:18 UTC (rev 209795)
</span><span class="lines">@@ -1,58 +0,0 @@
</span><del>-/*
- * Copyright (C) 2016 Apple Inc. All rights reserved.
- *
- * 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
-
-#include <wtf/PrintStream.h>
-#include <wtf/RecursiveLockAdapter.h>
-#include <wtf/WordLock.h>
-
-namespace WTF {
-
-// Makes every call to print() atomic.
-class LockedPrintStream : public PrintStream {
-public:
- LockedPrintStream(std::unique_ptr<PrintStream> target);
- virtual ~LockedPrintStream();
-
- void vprintf(const char* format, va_list) override WTF_ATTRIBUTE_PRINTF(2, 0);
- void flush() override;
-
-protected:
- PrintStream& begin() override;
- void end() override;
-
-private:
- // This needs to be a recursive lock because a printInternal or dump method could assert,
- // and that assert might want to log. Better to let it. This needs to be a WordLock so that
- // LockedPrintStream (i.e. cataLog) can be used to debug ParkingLot and Lock.
- RecursiveLockAdapter<WordLock> m_lock;
- std::unique_ptr<PrintStream> m_target;
-};
-
-} // namespace WTF
-
-using WTF::LockedPrintStream;
-
</del></span></pre></div>
<a id="trunkSourceWTFwtfPrintStreamcpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WTF/wtf/PrintStream.cpp (209794 => 209795)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WTF/wtf/PrintStream.cpp 2016-12-14 08:23:23 UTC (rev 209794)
+++ trunk/Source/WTF/wtf/PrintStream.cpp 2016-12-14 09:35:18 UTC (rev 209795)
</span><span class="lines">@@ -44,39 +44,10 @@
</span><span class="cx"> va_end(argList);
</span><span class="cx"> }
</span><span class="cx">
</span><del>-void PrintStream::printfVariableFormat(const char* format, ...)
-{
-#if COMPILER(CLANG)
-#pragma clang diagnostic push
-#pragma clang diagnostic ignored "-Wformat-nonliteral"
-#elif COMPILER(GCC)
-#pragma GCC diagnostic push
-#pragma GCC diagnostic ignored "-Wsuggest-attribute=format"
-#endif
- va_list argList;
- va_start(argList, format);
- vprintf(format, argList);
- va_end(argList);
-#if COMPILER(CLANG)
-#pragma clang diagnostic pop
-#elif COMPILER(GCC)
-#pragma GCC diagnostic pop
-#endif
-}
-
</del><span class="cx"> void PrintStream::flush()
</span><span class="cx"> {
</span><span class="cx"> }
</span><span class="cx">
</span><del>-PrintStream& PrintStream::begin()
-{
- return *this;
-}
-
-void PrintStream::end()
-{
-}
-
</del><span class="cx"> void printInternal(PrintStream& out, const char* string)
</span><span class="cx"> {
</span><span class="cx"> out.printf("%s", string);
</span></span></pre></div>
<a id="trunkSourceWTFwtfPrintStreamh"></a>
<div class="modfile"><h4>Modified: trunk/Source/WTF/wtf/PrintStream.h (209794 => 209795)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WTF/wtf/PrintStream.h 2016-12-14 08:23:23 UTC (rev 209794)
+++ trunk/Source/WTF/wtf/PrintStream.h 2016-12-14 09:35:18 UTC (rev 209795)
</span><span class="lines">@@ -30,7 +30,6 @@
</span><span class="cx"> #include <stdarg.h>
</span><span class="cx"> #include <wtf/FastMalloc.h>
</span><span class="cx"> #include <wtf/Noncopyable.h>
</span><del>-#include <wtf/Optional.h>
</del><span class="cx"> #include <wtf/RawPointer.h>
</span><span class="cx"> #include <wtf/RefPtr.h>
</span><span class="cx"> #include <wtf/StdLibExtras.h>
</span><span class="lines">@@ -51,7 +50,6 @@
</span><span class="cx"> virtual ~PrintStream();
</span><span class="cx">
</span><span class="cx"> WTF_EXPORT_PRIVATE void printf(const char* format, ...) WTF_ATTRIBUTE_PRINTF(2, 3);
</span><del>- WTF_EXPORT_PRIVATE void printfVariableFormat(const char* format, ...);
</del><span class="cx"> virtual void vprintf(const char* format, va_list) WTF_ATTRIBUTE_PRINTF(2, 0) = 0;
</span><span class="cx">
</span><span class="cx"> // Typically a no-op for many subclasses of PrintStream, this is a hint that
</span><span class="lines">@@ -58,40 +56,27 @@
</span><span class="cx"> // the implementation should flush its buffers if it had not done so already.
</span><span class="cx"> virtual void flush();
</span><span class="cx">
</span><del>- template<typename Func>
- void atomically(const Func& func)
</del><ins>+ void print() { }
+
+ template<typename T>
+ void print(const T& value)
</ins><span class="cx"> {
</span><del>- func(begin());
- end();
</del><ins>+ printInternal(*this, value);
</ins><span class="cx"> }
</span><del>-
- template<typename... Types>
- void print(const Types&... values)
</del><ins>+
+ template<typename T, typename... Types>
+ void print(const T& value, const Types&... remainingValues)
</ins><span class="cx"> {
</span><del>- atomically(
- [&] (PrintStream& out) {
- out.printImpl(values...);
- });
</del><ins>+ printInternal(*this, value);
+ print(remainingValues...);
</ins><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> template<typename... Types>
</span><span class="cx"> void println(const Types&... values)
</span><span class="cx"> {
</span><del>- print(values..., "\n");
</del><ins>+ print(values...);
+ print("\n");
</ins><span class="cx"> }
</span><del>-
-protected:
- void printImpl() { }
-
- template<typename T, typename... Types>
- void printImpl(const T& value, const Types&... remainingValues)
- {
- printInternal(*this, value);
- printImpl(remainingValues...);
- }
-
- virtual PrintStream& begin();
- virtual void end();
</del><span class="cx"> };
</span><span class="cx">
</span><span class="cx"> WTF_EXPORT_PRIVATE void printInternal(PrintStream&, const char*);
</span><span class="lines">@@ -281,66 +266,11 @@
</span><span class="cx"> return ValueIgnoringContext<T, U>(value);
</span><span class="cx"> }
</span><span class="cx">
</span><del>-template<unsigned index, typename... Types>
-struct FormatImplUnpacker {
- template<typename... Args>
- static void unpack(PrintStream& out, const std::tuple<Types...>& tuple, const Args&... values);
-};
-
-template<typename... Types>
-struct FormatImplUnpacker<0, Types...> {
- template<typename... Args>
- static void unpack(PrintStream& out, const std::tuple<Types...>& tuple, const Args&... values)
- {
- out.printfVariableFormat(std::get<0>(tuple), values...);
- }
-};
-
-template<unsigned index, typename... Types>
-template<typename... Args>
-void FormatImplUnpacker<index, Types...>::unpack(PrintStream& out, const std::tuple<Types...>& tuple, const Args&... values)
-{
- FormatImplUnpacker<index - 1, Types...>::unpack(out, tuple, std::get<index>(tuple), values...);
-}
-
-template<typename... Types>
-class FormatImpl {
-public:
- FormatImpl(Types... values)
- : m_values(values...)
- {
- }
-
- void dump(PrintStream& out) const
- {
- FormatImplUnpacker<sizeof...(Types) - 1, Types...>::unpack(out, m_values);
- }
-
-private:
- std::tuple<Types...> m_values;
-};
-
-template<typename... Types>
-FormatImpl<Types...> format(Types... values)
-{
- return FormatImpl<Types...>(values...);
-}
-
-template<typename T>
-void printInternal(PrintStream& out, const std::optional<T>& value)
-{
- if (value)
- out.print(*value);
- else
- out.print("<nullopt>");
-}
-
</del><span class="cx"> } // namespace WTF
</span><span class="cx">
</span><span class="cx"> using WTF::CharacterDump;
</span><span class="cx"> using WTF::PointerDump;
</span><span class="cx"> using WTF::PrintStream;
</span><del>-using WTF::format;
</del><span class="cx"> using WTF::ignoringContext;
</span><span class="cx"> using WTF::inContext;
</span><span class="cx"> using WTF::pointerDump;
</span></span></pre></div>
<a id="trunkSourceWTFwtfRecursiveLockAdapterh"></a>
<div class="delfile"><h4>Deleted: trunk/Source/WTF/wtf/RecursiveLockAdapter.h (209794 => 209795)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WTF/wtf/RecursiveLockAdapter.h 2016-12-14 08:23:23 UTC (rev 209794)
+++ trunk/Source/WTF/wtf/RecursiveLockAdapter.h 2016-12-14 09:35:18 UTC (rev 209795)
</span><span class="lines">@@ -1,93 +0,0 @@
</span><del>-/*
- * Copyright (C) 2016 Apple Inc. All rights reserved.
- *
- * 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
-
-#include <wtf/Threading.h>
-
-namespace WTF {
-
-template<typename LockType>
-class RecursiveLockAdapter {
-public:
- RecursiveLockAdapter()
- {
- }
-
- void lock()
- {
- ThreadIdentifier me = currentThread();
- if (me == m_owner) {
- m_recursionCount++;
- return;
- }
-
- m_lock.lock();
- ASSERT(!m_owner);
- ASSERT(!m_recursionCount);
- m_owner = me;
- m_recursionCount = 1;
- }
-
- void unlock()
- {
- if (--m_recursionCount)
- return;
- m_owner = 0;
- m_lock.unlock();
- }
-
- bool tryLock()
- {
- ThreadIdentifier me = currentThread();
- if (me == m_owner) {
- m_recursionCount++;
- return true;
- }
-
- if (!m_lock.tryLock())
- return false;
-
- ASSERT(!m_owner);
- ASSERT(!m_recursionCount);
- m_owner = me;
- m_recursionCount = 1;
- return true;
- }
-
- bool isLocked() const
- {
- return m_lock.isLocked();
- }
-
-private:
- ThreadIdentifier m_owner { 0 };
- unsigned m_recursionCount { 0 };
- LockType m_lock;
-};
-
-} // namespace WTF
-
-
</del></span></pre></div>
<a id="trunkSourceWTFwtfWordLockcpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WTF/wtf/WordLock.cpp (209794 => 209795)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WTF/wtf/WordLock.cpp 2016-12-14 08:23:23 UTC (rev 209794)
+++ trunk/Source/WTF/wtf/WordLock.cpp 2016-12-14 09:35:18 UTC (rev 209795)
</span><span class="lines">@@ -26,6 +26,8 @@
</span><span class="cx"> #include "config.h"
</span><span class="cx"> #include "WordLock.h"
</span><span class="cx">
</span><ins>+#include "DataLog.h"
+#include "StringPrintStream.h"
</ins><span class="cx"> #include "ThreadSpecific.h"
</span><span class="cx"> #include "ThreadingPrimitives.h"
</span><span class="cx"> #include <condition_variable>
</span></span></pre></div>
<a id="trunkSourceWTFwtfWordLockh"></a>
<div class="modfile"><h4>Modified: trunk/Source/WTF/wtf/WordLock.h (209794 => 209795)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WTF/wtf/WordLock.h 2016-12-14 08:23:23 UTC (rev 209794)
+++ trunk/Source/WTF/wtf/WordLock.h 2016-12-14 09:35:18 UTC (rev 209795)
</span><span class="lines">@@ -42,10 +42,6 @@
</span><span class="cx"> // Lock instead. WordLock sits lower in the stack and is used to implement Lock, so Lock is the main
</span><span class="cx"> // client of WordLock.
</span><span class="cx">
</span><del>-// NOTE: This is also a great lock to use if you are very low in the stack. For example,
-// PrintStream uses this so that ParkingLot and Lock can use PrintStream. This means that if you
-// try to use dataLog to debug this code, you will have a bad time.
-
</del><span class="cx"> struct WordLockBase {
</span><span class="cx"> void lock()
</span><span class="cx"> {
</span></span></pre>
</div>
</div>
</body>
</html>