Title: [196981] trunk/PerformanceTests
Revision
196981
Author
[email protected]
Date
2016-02-23 09:42:12 -0800 (Tue, 23 Feb 2016)

Log Message

Unreviewed change to revert extraneous changes made part of change set 196955.

* MallocBench/MallocBench/Interpreter.cpp:
(Interpreter::doMallocOp):
(Interpreter::Thread::switchTo):
(writeData): Deleted.

Modified Paths

Diff

Modified: trunk/PerformanceTests/ChangeLog (196980 => 196981)


--- trunk/PerformanceTests/ChangeLog	2016-02-23 17:40:20 UTC (rev 196980)
+++ trunk/PerformanceTests/ChangeLog	2016-02-23 17:42:12 UTC (rev 196981)
@@ -1,3 +1,12 @@
+2016-02-23  Michael Saboff  <[email protected]>
+
+        Unreviewed change to revert extraneous changes made part of change set 196955.
+
+        * MallocBench/MallocBench/Interpreter.cpp:
+        (Interpreter::doMallocOp):
+        (Interpreter::Thread::switchTo):
+        (writeData): Deleted.
+
 2016-02-19  Michael Saboff  <[email protected]>
 
         MallocBench: Added recording for nimlang website, new recording details and added new options

Modified: trunk/PerformanceTests/MallocBench/MallocBench/Interpreter.cpp (196980 => 196981)


--- trunk/PerformanceTests/MallocBench/MallocBench/Interpreter.cpp	2016-02-23 17:40:20 UTC (rev 196980)
+++ trunk/PerformanceTests/MallocBench/MallocBench/Interpreter.cpp	2016-02-23 17:42:12 UTC (rev 196981)
@@ -94,7 +94,6 @@
     m_opsCursor = m_opsInBuffer = 0;
     doOnSameThread(0);
 
-    fprintf(stderr, "Done running\n");
     for (auto thread : m_threads)
         thread->stop();
 
@@ -223,30 +222,13 @@
     return result;
 }
 
-static void writeData(void* start, size_t size)
-{
-    char* writePtr = reinterpret_cast<char*>(start);
-
-    for (size_t sizeLeft = size; !!sizeLeft; ) {
-        size_t sizeThisIter = std::min(sizeLeft, 4096ul);
-        
-        writePtr[0] = random() & 0xff;
-        writePtr[1] = random() & 0xff;
-        writePtr[2] = random() & 0xff;
-        writePtr[3] = random() & 0xff;
-
-        writePtr += sizeThisIter;
-        sizeLeft -= sizeThisIter;
-    }
-}
-
 void Interpreter::doMallocOp(Op op, ThreadId threadId)
 {
     switch (op.opcode) {
         case op_malloc: {
             m_objects[op.slot] = { mbmalloc(op.size), op.size };
             assert(m_objects[op.slot].object);
-            writeData(m_objects[op.slot].object, op.size);
+            bzero(m_objects[op.slot].object, op.size);
             break;
         }
         case op_free: {
@@ -266,7 +248,7 @@
             size_t alignment = compute2toPower(op.alignLog2);
             m_objects[op.slot] = { mbmemalign(alignment, op.size), op.size };
             assert(m_objects[op.slot].object);
-            writeData(m_objects[op.slot].object, op.size);
+            bzero(m_objects[op.slot].object, op.size);
             break;
         }
         default: {
@@ -312,4 +294,4 @@
     std::unique_lock<std::mutex> lock(m_myInterpreter->m_threadMutex);
     m_myInterpreter->m_currentThreadId = m_threadId;
     m_shouldRun.notify_one();
-}
\ No newline at end of file
+}
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to