Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(332)

Issue 256024: Set JEMalloc as the default allocator (instead of TCMalloc)... (Closed)

Created:
11 years, 2 months ago by jar (doing other things)
Modified:
9 years, 7 months ago
Reviewers:
CC:
chromium-reviews_googlegroups.com
Visibility:
Public.

Description

Set JEMalloc as the default allocator (instead of TCMalloc) I got this change approved to try out earlier, and landed it. I rolled it back (to be conservative) after I got a chromebot run started. Since that bot run didn't show any giant problems, this will re-land the change, so we can see how users (such as on a dev channel release) do with this allocator. This is really experimental... but it will give us some info to make a better decision between ongoing tuning of TCMalloc and at least temporarilly using JEMalloc. tbr=mbelshe Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=27708

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M third_party/tcmalloc/allocator_shim.cc View 1 chunk +1 line, -1 line 0 comments Download

Powered by Google App Engine
This is Rietveld 408576698