Changeset 159508 in webkit for trunk/Source/JavaScriptCore/heap/IncrementalSweeper.cpp

Timestamp:

Nov 19, 2013, 11:10:56 AM (12 years ago)

Author:

[email protected]

Message:

IncrementalSweeper needs to use DelayedReleaseScope too
​https://bugs.webkit.org/show_bug.cgi?id=124558

Reviewed by Filip Pizlo.

It does sweeping too, so it needs to use it. Also refactored an
ASSERT that should have caught this sooner.

• heap/DelayedReleaseScope.h:

(JSC::DelayedReleaseScope::isInEffectFor):

• heap/IncrementalSweeper.cpp:

(JSC::IncrementalSweeper::doSweep):

• heap/MarkedBlock.cpp:

(JSC::MarkedBlock::sweep):

• heap/MarkedSpace.cpp:

(JSC::MarkedSpace::sweep):

File:

• trunk/Source/JavaScriptCore/heap/IncrementalSweeper.cpp (modified) (2 diffs)

trunk/Source/JavaScriptCore/heap/IncrementalSweeper.cpp

@@ -28 +28 @@
 
 #include "APIShims.h"
+#include "DelayedReleaseScope.h"
 #include "Heap.h"
 #include "JSObject.h"
@@ -101 +102 @@
 void IncrementalSweeper::doSweep(double sweepBeginTime)
 {
+    DelayedReleaseScope scope(m_vm->heap.m_objectSpace);
     while (m_currentBlockToSweepIndex < m_blocksToSweep.size()) {
         sweepNextBlock();