Title: [199756] trunk/Source/bmalloc
Revision
199756
Author
[email protected]
Date
2016-04-19 20:08:33 -0700 (Tue, 19 Apr 2016)

Log Message

2016-04-19  Geoffrey Garen  <[email protected]>

        Unreviewed, try to fix an ASSERT seen on the bots.

        * bmalloc/Heap.cpp:
        (bmalloc::Heap::tryAllocateLarge): This ASSERT is supposed to be about
        alignment, not size. Oops.

Modified Paths

Diff

Modified: trunk/Source/bmalloc/ChangeLog (199755 => 199756)


--- trunk/Source/bmalloc/ChangeLog	2016-04-20 02:41:00 UTC (rev 199755)
+++ trunk/Source/bmalloc/ChangeLog	2016-04-20 03:08:33 UTC (rev 199756)
@@ -1,5 +1,13 @@
 2016-04-19  Geoffrey Garen  <[email protected]>
 
+        Unreviewed, try to fix an ASSERT seen on the bots.
+
+        * bmalloc/Heap.cpp:
+        (bmalloc::Heap::tryAllocateLarge): This ASSERT is supposed to be about
+        alignment, not size. Oops.
+
+2016-04-19  Geoffrey Garen  <[email protected]>
+
         bmalloc: Merge the large and xlarge allocators
         https://bugs.webkit.org/show_bug.cgi?id=156734
 

Modified: trunk/Source/bmalloc/bmalloc/Heap.cpp (199755 => 199756)


--- trunk/Source/bmalloc/bmalloc/Heap.cpp	2016-04-20 02:41:00 UTC (rev 199755)
+++ trunk/Source/bmalloc/bmalloc/Heap.cpp	2016-04-20 03:08:33 UTC (rev 199756)
@@ -343,7 +343,7 @@
 void* Heap::tryAllocateLarge(std::lock_guard<StaticMutex>& lock, size_t alignment, size_t size)
 {
     BASSERT(size <= largeMax);
-    BASSERT(size <= largeMax / 2);
+    BASSERT(alignment <= largeMax / 2);
     BASSERT(isPowerOfTwo(alignment));
 
     size = size ? roundUpToMultipleOf(largeAlignment, size) : largeAlignment;
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to