JavaScriptCore-7600.1.4.9.tar.gz

[apple/javascriptcore.git] / heap / HandleSet.cpp

diff --git a/heap/HandleSet.cpp b/heap/HandleSet.cpp
index a6ccf29eb4649ad40185d61363f1bce397260e66..be667e8202fc9c0edd6e3e4493331c20db06f1e1 100644 (file)
--- a/heap/HandleSet.cpp
+++ b/heap/HandleSet.cpp
@@ -26,24 +26,35 @@
 #include "config.h"
 #include "HandleSet.h"
 
 #include "config.h"
 #include "HandleSet.h"
 

+#include "HandleBlock.h"
+#include "HandleBlockInlines.h"

 #include "HeapRootVisitor.h"
 #include "JSObject.h"
 #include "HeapRootVisitor.h"
 #include "JSObject.h"

+#include "JSCInlines.h"
+#include <wtf/DataLog.h>

 
 namespace JSC {
 
 
 namespace JSC {
 

-HandleSet::HandleSet(JSGlobalData* globalData)
-    : m_globalData(globalData)
-    , m_nextToFinalize(0)
+HandleSet::HandleSet(VM* vm)
+    : m_vm(vm)

 {
     grow();
 }
 
 {
     grow();
 }
 

+HandleSet::~HandleSet()
+{
+    while (!m_blockList.isEmpty())
+        m_vm->heap.blockAllocator().deallocate(HandleBlock::destroy(m_blockList.removeHead()));
+}
+

 void HandleSet::grow()
 {
 void HandleSet::grow()
 {

-    Node* block = m_blockStack.grow();
-    for (int i = m_blockStack.blockLength - 1; i >= 0; --i) {
-        Node* node = &block[i];
-        new (NotNull, node) Node(this);
+    HandleBlock* newBlock = HandleBlock::create(m_vm->heap.blockAllocator().allocate<HandleBlock>(), this);
+    m_blockList.append(newBlock);
+
+    for (int i = newBlock->nodeCapacity() - 1; i >= 0; --i) {
+        Node* node = newBlock->nodeAtIndex(i);
+        new (NotNull, node) Node;

         m_freeList.push(node);
     }
 }
         m_freeList.push(node);
     }
 }


@@ -53,8 +64,7 @@ void HandleSet::visitStrongHandles(HeapRootVisitor& heapRootVisitor)
     Node* end = m_strongList.end();
     for (Node* node = m_strongList.begin(); node != end; node = node->next()) {
 #if ENABLE(GC_VALIDATION)
     Node* end = m_strongList.end();
     for (Node* node = m_strongList.begin(); node != end; node = node->next()) {
 #if ENABLE(GC_VALIDATION)

-        if (!isLiveNode(node))
-            CRASH();
+        RELEASE_ASSERT(isLiveNode(node));

 #endif
         heapRootVisitor.visit(node->slot());
     }
 #endif
         heapRootVisitor.visit(node->slot());
     }


@@ -62,18 +72,12 @@ void HandleSet::visitStrongHandles(HeapRootVisitor& heapRootVisitor)
 
 void HandleSet::writeBarrier(HandleSlot slot, const JSValue& value)
 {
 
 void HandleSet::writeBarrier(HandleSlot slot, const JSValue& value)
 {

-    // Forbid assignment to handles during the finalization phase, since it would violate many GC invariants.
-    // File a bug with stack trace if you hit this.
-    if (m_nextToFinalize)
-        CRASH();
-

     if (!value == !*slot && slot->isCell() == value.isCell())
         return;
 
     Node* node = toNode(slot);
 #if ENABLE(GC_VALIDATION)
     if (!value == !*slot && slot->isCell() == value.isCell())
         return;
 
     Node* node = toNode(slot);
 #if ENABLE(GC_VALIDATION)

-    if (!isLiveNode(node))
-        CRASH();
+    RELEASE_ASSERT(isLiveNode(node));

 #endif
     SentinelLinkedList<Node>::remove(node);
     if (!value || !value.isCell()) {
 #endif
     SentinelLinkedList<Node>::remove(node);
     if (!value || !value.isCell()) {


@@ -83,8 +87,7 @@ void HandleSet::writeBarrier(HandleSlot slot, const JSValue& value)
 
     m_strongList.push(node);
 #if ENABLE(GC_VALIDATION)
 
     m_strongList.push(node);
 #if ENABLE(GC_VALIDATION)

-    if (!isLiveNode(node))
-        CRASH();
+    RELEASE_ASSERT(isLiveNode(node));

 #endif
 }
 
 #endif
 }