A few MallocBench record/replay fixes
authorggaren@apple.com <ggaren@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 14 Apr 2014 19:11:50 +0000 (19:11 +0000)
committerggaren@apple.com <ggaren@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 14 Apr 2014 19:11:50 +0000 (19:11 +0000)
https://bugs.webkit.org/show_bug.cgi?id=131627

Reviewed by Andreas Kling.

* MallocBench/MallocBench/Interpreter.cpp:
(Interpreter::run): Accept 0-sized allocations without asserting because
WebKit does that sometimes.

* MallocBench/MallocBench/flickr.ops:
* MallocBench/MallocBench/flickr_memory_warning.ops:
* MallocBench/MallocBench/reddit.ops:
* MallocBench/MallocBench/reddit_memory_warning.ops:
* MallocBench/MallocBench/theverge.ops:
* MallocBench/MallocBench/theverge_memory_warning.ops: Updated these
recordings because a bug in the recording mechanism caused one out of
every few thousand slot values to be bogus.

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

PerformanceTests/ChangeLog
PerformanceTests/MallocBench/MallocBench/Interpreter.cpp
PerformanceTests/MallocBench/MallocBench/flickr.ops
PerformanceTests/MallocBench/MallocBench/flickr_memory_warning.ops
PerformanceTests/MallocBench/MallocBench/reddit.ops
PerformanceTests/MallocBench/MallocBench/reddit_memory_warning.ops
PerformanceTests/MallocBench/MallocBench/theverge.ops
PerformanceTests/MallocBench/MallocBench/theverge_memory_warning.ops

index ce64fd5..1effdf8 100644 (file)
@@ -1,3 +1,23 @@
+2014-04-14  Geoffrey Garen  <ggaren@apple.com>
+
+        A few MallocBench record/replay fixes
+        https://bugs.webkit.org/show_bug.cgi?id=131627
+
+        Reviewed by Andreas Kling.
+
+        * MallocBench/MallocBench/Interpreter.cpp:
+        (Interpreter::run): Accept 0-sized allocations without asserting because
+        WebKit does that sometimes.
+
+        * MallocBench/MallocBench/flickr.ops:
+        * MallocBench/MallocBench/flickr_memory_warning.ops:
+        * MallocBench/MallocBench/reddit.ops:
+        * MallocBench/MallocBench/reddit_memory_warning.ops:
+        * MallocBench/MallocBench/theverge.ops:
+        * MallocBench/MallocBench/theverge_memory_warning.ops: Updated these
+        recordings because a bug in the recording mechanism caused one out of
+        every few thousand slot values to be bogus.
+
 2014-04-13  Geoffrey Garen  <ggaren@apple.com>
 
         Added some website recordings to MallocBench -- taken from Membuster
index 9178b47..92636ca 100644 (file)
@@ -101,14 +101,12 @@ void Interpreter::run()
             }
             case op_free: {
                 assert(m_objects[op.slot].object);
-                assert(m_objects[op.slot].size);
                 mbfree(m_objects[op.slot].object, m_objects[op.slot].size);
                 m_objects[op.slot] = { 0, 0 };
                 break;
             }
             case op_realloc: {
                 assert(m_objects[op.slot].object);
-                assert(m_objects[op.slot].size);
                 m_objects[op.slot] = { mbrealloc(m_objects[op.slot].object, m_objects[op.slot].size, op.size), op.size };
                 break;
             }
index 9ea1499..f78b430 100644 (file)
Binary files a/PerformanceTests/MallocBench/MallocBench/flickr.ops and b/PerformanceTests/MallocBench/MallocBench/flickr.ops differ
index 58438c9..2e59f91 100644 (file)
Binary files a/PerformanceTests/MallocBench/MallocBench/flickr_memory_warning.ops and b/PerformanceTests/MallocBench/MallocBench/flickr_memory_warning.ops differ
index ccf25f7..b8d9a71 100644 (file)
Binary files a/PerformanceTests/MallocBench/MallocBench/reddit.ops and b/PerformanceTests/MallocBench/MallocBench/reddit.ops differ
index 7288375..7048efb 100644 (file)
Binary files a/PerformanceTests/MallocBench/MallocBench/reddit_memory_warning.ops and b/PerformanceTests/MallocBench/MallocBench/reddit_memory_warning.ops differ
index fcd364c..d73fbb4 100644 (file)
Binary files a/PerformanceTests/MallocBench/MallocBench/theverge.ops and b/PerformanceTests/MallocBench/MallocBench/theverge.ops differ
index 88ac20e..9a8d12d 100644 (file)
Binary files a/PerformanceTests/MallocBench/MallocBench/theverge_memory_warning.ops and b/PerformanceTests/MallocBench/MallocBench/theverge_memory_warning.ops differ