Changeset 102261 in webkit for trunk/Source/JavaScriptCore/heap/Heap.cpp

Timestamp:

Dec 7, 2011, 11:52:01 AM (13 years ago)

Author:

[email protected]

Message:

Non-Mac devices should benefit from a larger heap
​https://bugs.webkit.org/show_bug.cgi?id=74015

Reviewed by Geoff Garen.

Removed the ENABLE(LARGE_HEAP) option from Platform.h, since it was only used in
Heap.cpp, and got in the way of having more granular, per-platform control over
what the heap size should be. Bumped the heap size to 8MB on iOS (was 512KB).

• heap/Heap.cpp:

(JSC::GCTimer::heapSizeForHint):

• wtf/Platform.h:

File:

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

trunk/Source/JavaScriptCore/heap/Heap.cpp

@@ -43 +43 @@
 namespace { 
 
+#if CPU(X86) || CPU(X86_64)
 static const size_t largeHeapSize = 16 * 1024 * 1024;
+#elif PLATFORM(IOS)
+static const size_t largeHeapSize = 8 * 1024 * 1024;
+#else
+static const size_t largeHeapSize = 512 * 1024;
+#endif
 static const size_t smallHeapSize = 512 * 1024;
 
@@ -142 +148 @@
 static size_t heapSizeForHint(HeapSize heapSize)
 {
-#if ENABLE(LARGE_HEAP)
     if (heapSize == LargeHeap)
         return largeHeapSize;
     ASSERT(heapSize == SmallHeap);
     return smallHeapSize;
-#else
-    ASSERT_UNUSED(heapSize, heapSize == LargeHeap || heapSize == SmallHeap);
-    return smallHeapSize;
-#endif
 }