From 9fc893add031bb2c8b82ae8539115815575bdc8c Mon Sep 17 00:00:00 2001 From: Mike Hommey Date: Thu, 23 Nov 2017 15:36:13 +0900 Subject: [PATCH] Bug 1420092 - Enable mozjemalloc by default on 32-bit OSX. r=njn Hopefully, the bug we worked around by disabling jemalloc on 32-bit OSX is gone. We're not shipping 32-bit binaries for OSX anyways. --HG-- extra : rebase_source : 148a80a7ab006d3be81fb931cbbb4ad2c81690c3 --- build/moz.configure/memory.configure | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/build/moz.configure/memory.configure b/build/moz.configure/memory.configure index 236070bda202..9105387b1623 100644 --- a/build/moz.configure/memory.configure +++ b/build/moz.configure/memory.configure @@ -22,16 +22,12 @@ def jemalloc(value, target, build_project, c_compiler): if build_project == 'js': return True - if target.kernel == 'Darwin' and target.cpu == 'x86_64': - # Don't enable by default on 32-bits OSX. See bug 702250. + if target.kernel in ('Darwin', 'Linux'): return True if target.kernel == 'WINNT' and c_compiler.type in ('msvc', 'clang-cl'): return True - if target.kernel == 'Linux': - return True - if value and target.kernel not in ('WINNT', 'Linux', 'Darwin', 'kFreeBSD', 'FreeBSD', 'NetBSD'): die('--enable-jemalloc is not supported on %s', target.kernel)