No bug - Remove the TODO item that was implemented in bug 1300948. r=me
authorMike Hommey <mh+mozilla@glandium.org>
Tue, 05 Sep 2017 17:28:41 +0900
changeset 429764 ec45cf34da3c7580be94b263f0a7e9671f3c347e
parent 429763 5b048319e83005e1e95d22fac5a548626cd87eb3
child 429765 b21ad1316ba60537d00e99fa89951580802c963b
push id7761
push userjlund@mozilla.com
push dateFri, 15 Sep 2017 00:19:52 +0000
treeherdermozilla-beta@c38455951db4 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersme
bugs1300948
milestone57.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
No bug - Remove the TODO item that was implemented in bug 1300948. r=me
memory/replace/logalloc/LogAlloc.cpp
--- a/memory/replace/logalloc/LogAlloc.cpp
+++ b/memory/replace/logalloc/LogAlloc.cpp
@@ -164,20 +164,16 @@ replace_get_bridge()
 {
   static LogAllocBridge bridge;
   return &bridge;
 }
 
 /* Do a simple, text-form, log of all calls to replace-malloc functions.
  * Use locking to guarantee that an allocation that did happen is logged
  * before any other allocation/free happens.
- * TODO: Add a thread id to the log: different allocators, or even different
- * configurations of jemalloc behave differently when allocations are coming
- * from different threads. Reproducing those multi-threaded workloads would be
- * useful to test those differences.
  */
 
 void*
 replace_malloc(size_t aSize)
 {
   AutoLock lock(sLock);
   void* ptr = sFuncs->malloc(aSize);
   if (ptr) {