Remove Heap::isLive()
authorfpizlo@apple.com <fpizlo@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 13 Sep 2016 23:19:48 +0000 (23:19 +0000)
committerfpizlo@apple.com <fpizlo@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 13 Sep 2016 23:19:48 +0000 (23:19 +0000)
https://bugs.webkit.org/show_bug.cgi?id=161933

Reviewed by Mark Lam.

Before I put any more effort into maintaining this weird function, I decided to check how it
was used. It turns out it's not.

* heap/Heap.h:
* heap/HeapInlines.h:
(JSC::Heap::isLive): Deleted.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@205886 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/JavaScriptCore/ChangeLog
Source/JavaScriptCore/heap/Heap.h
Source/JavaScriptCore/heap/HeapInlines.h

index 9b9952d..78bd5f3 100644 (file)
@@ -1,3 +1,17 @@
+2016-09-13  Filip Pizlo  <fpizlo@apple.com>
+
+        Remove Heap::isLive()
+        https://bugs.webkit.org/show_bug.cgi?id=161933
+
+        Reviewed by Mark Lam.
+        
+        Before I put any more effort into maintaining this weird function, I decided to check how it
+        was used. It turns out it's not.
+
+        * heap/Heap.h:
+        * heap/HeapInlines.h:
+        (JSC::Heap::isLive): Deleted.
+
 2016-09-13  Mark Lam  <mark.lam@apple.com>
 
         DFG NewArrayBuffer node should watch for "have a bad time" state change.
 2016-09-13  Mark Lam  <mark.lam@apple.com>
 
         DFG NewArrayBuffer node should watch for "have a bad time" state change.
index 81360dc..5161792 100644 (file)
@@ -97,7 +97,6 @@ public:
     // our scan to run faster. 
     static const unsigned s_timeCheckResolution = 16;
 
     // our scan to run faster. 
     static const unsigned s_timeCheckResolution = 16;
 
-    static bool isLive(const void*);
     static bool isMarked(const void*);
     static bool isMarkedConcurrently(const void*);
     static bool testAndSetMarked(HeapVersion, const void*);
     static bool isMarked(const void*);
     static bool isMarkedConcurrently(const void*);
     static bool testAndSetMarked(HeapVersion, const void*);
index 17725f9..6c4d0b4 100644 (file)
@@ -74,17 +74,6 @@ inline Heap* Heap::heap(const JSValue v)
     return heap(v.asCell());
 }
 
     return heap(v.asCell());
 }
 
-inline bool Heap::isLive(const void* rawCell)
-{
-    ASSERT(!mayBeGCThread());
-    HeapCell* cell = bitwise_cast<HeapCell*>(rawCell);
-    if (cell->isLargeAllocation())
-        return cell->largeAllocation().isLive();
-    MarkedBlock& block = cell->markedBlock();
-    block.flipIfNecessary(block.vm()->heap.objectSpace().version());
-    return block.handle().isLiveCell(cell);
-}
-
 ALWAYS_INLINE bool Heap::isMarked(const void* rawCell)
 {
     ASSERT(!mayBeGCThread());
 ALWAYS_INLINE bool Heap::isMarked(const void* rawCell)
 {
     ASSERT(!mayBeGCThread());