From a910a498a81b76fe4ed171728c52a6425a0ae48a Mon Sep 17 00:00:00 2001
From: Jimmy Chen <jchen.tc@gmail.com>
Date: Thu, 27 Jun 2024 17:11:22 +1000
Subject: [PATCH] Enable jemalloc by default on non windows targets (#5995)

* Enable jemalloc by default on non windows target.

* Update `allocator_name` function to check for `target_os` instead as we've deprecated `jemalloc` feature.
---
 Makefile               |  8 --------
 lighthouse/Cargo.toml  | 10 ++++++++--
 lighthouse/src/main.rs |  6 +++---
 3 files changed, 11 insertions(+), 13 deletions(-)

diff --git a/Makefile b/Makefile
index c3c8d13238..3f8e688df1 100644
--- a/Makefile
+++ b/Makefile
@@ -14,14 +14,6 @@ BUILD_PATH_AARCH64 = "target/$(AARCH64_TAG)/release"
 PINNED_NIGHTLY ?= nightly
 CLIPPY_PINNED_NIGHTLY=nightly-2022-05-19
 
-# List of features to use when building natively. Can be overridden via the environment.
-# No jemalloc on Windows
-ifeq ($(OS),Windows_NT)
-    FEATURES?=
-else
-    FEATURES?=jemalloc
-endif
-
 # List of features to use when cross-compiling. Can be overridden via the environment.
 CROSS_FEATURES ?= gnosis,slasher-lmdb,slasher-mdbx,jemalloc
 
diff --git a/lighthouse/Cargo.toml b/lighthouse/Cargo.toml
index 20466b5de7..64b08b113e 100644
--- a/lighthouse/Cargo.toml
+++ b/lighthouse/Cargo.toml
@@ -22,8 +22,14 @@ gnosis = []
 slasher-mdbx = ["slasher/mdbx"]
 # Support slasher LMDB backend.
 slasher-lmdb = ["slasher/lmdb"]
-# Use jemalloc.
-jemalloc = ["malloc_utils/jemalloc"]
+# Deprecated. This is now enabled by default on non windows targets.
+jemalloc = []
+
+[target.'cfg(not(target_os = "windows"))'.dependencies]
+malloc_utils = { workspace = true, features = ["jemalloc"] }
+
+[target.'cfg(target_os = "windows")'.dependencies]
+malloc_utils = { workspace = true }
 
 [dependencies]
 beacon_node = { workspace = true }
diff --git a/lighthouse/src/main.rs b/lighthouse/src/main.rs
index abee30737c..47b44d3828 100644
--- a/lighthouse/src/main.rs
+++ b/lighthouse/src/main.rs
@@ -51,10 +51,10 @@ fn bls_library_name() -> &'static str {
 }
 
 fn allocator_name() -> &'static str {
-    if cfg!(feature = "jemalloc") {
-        "jemalloc"
-    } else {
+    if cfg!(target_os = "windows") {
         "system"
+    } else {
+        "jemalloc"
     }
 }