Skip to content

Commit 67fe1a2

Browse files
committed
mimalloc: offer a build-time option to enable it
By defining `USE_MIMALLOC`, Git can now be compiled with that nicely-fast and small allocator. Note that we have to disable a couple `DEVELOPER` options to build mimalloc's source code, as it makes heavy use of declarations after statements, among other things that disagree with Git's conventions. We even have to silence some GCC warnings in non-DEVELOPER mode. For example, the `-Wno-array-bounds` flag is needed because in `-O2` builds, trying to call `NtCurrentTeb()` (which `_mi_thread_id()` does on Windows) causes the bogus warning about a system header, likely related to https://sourceforge.net/p/mingw-w64/mailman/message/37674519/ and to https://gcc.gnu.org/bugzilla/show_bug.cgi?id=99578: C:/git-sdk-64-minimal/mingw64/include/psdk_inc/intrin-impl.h:838:1: error: array subscript 0 is outside array bounds of 'long long unsigned int[0]' [-Werror=array-bounds] 838 | __buildreadseg(__readgsqword, unsigned __int64, "gs", "q") | ^~~~~~~~~~~~~~ Also: The `mimalloc` library uses C11-style atomics, therefore we must require that standard when compiling with GCC if we want to use `mimalloc` (instead of requiring "only" C99). This is what we do in the CMake definition already, therefore this commit does not need to touch `contrib/buildsystems/`. Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
1 parent 328fb26 commit 67fe1a2

File tree

5 files changed

+55
-1
lines changed

5 files changed

+55
-1
lines changed

Makefile

Lines changed: 41 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1492,6 +1492,7 @@ BUILTIN_OBJS += builtin/write-tree.o
14921492
# upstream unnecessarily (making merging in future changes easier).
14931493
THIRD_PARTY_SOURCES += compat/inet_ntop.c
14941494
THIRD_PARTY_SOURCES += compat/inet_pton.c
1495+
THIRD_PARTY_SOURCES += compat/mimalloc/%
14951496
THIRD_PARTY_SOURCES += compat/nedmalloc/%
14961497
THIRD_PARTY_SOURCES += compat/obstack.%
14971498
THIRD_PARTY_SOURCES += compat/poll/%
@@ -2215,6 +2216,46 @@ ifdef USE_NED_ALLOCATOR
22152216
OVERRIDE_STRDUP = YesPlease
22162217
endif
22172218

2219+
ifdef USE_MIMALLOC
2220+
MIMALLOC_OBJS = \
2221+
compat/mimalloc/alloc-aligned.o \
2222+
compat/mimalloc/alloc.o \
2223+
compat/mimalloc/arena.o \
2224+
compat/mimalloc/bitmap.o \
2225+
compat/mimalloc/heap.o \
2226+
compat/mimalloc/init.o \
2227+
compat/mimalloc/libc.o \
2228+
compat/mimalloc/options.o \
2229+
compat/mimalloc/os.o \
2230+
compat/mimalloc/page.o \
2231+
compat/mimalloc/random.o \
2232+
compat/mimalloc/prim/prim.o \
2233+
compat/mimalloc/segment.o \
2234+
compat/mimalloc/segment-map.o \
2235+
compat/mimalloc/stats.o
2236+
2237+
COMPAT_CFLAGS += -Icompat/mimalloc -DMI_DEBUG=0 -DUSE_MIMALLOC --std=gnu11
2238+
COMPAT_OBJS += $(MIMALLOC_OBJS)
2239+
2240+
$(MIMALLOC_OBJS): COMPAT_CFLAGS += -DBANNED_H
2241+
2242+
$(MIMALLOC_OBJS): COMPAT_CFLAGS += \
2243+
-DMI_WIN_USE_FLS \
2244+
-Wno-attributes \
2245+
-Wno-unknown-pragmas \
2246+
-Wno-unused-function \
2247+
-Wno-array-bounds
2248+
2249+
ifdef DEVELOPER
2250+
$(MIMALLOC_OBJS): COMPAT_CFLAGS += \
2251+
-Wno-pedantic \
2252+
-Wno-declaration-after-statement \
2253+
-Wno-old-style-definition \
2254+
-Wno-missing-prototypes \
2255+
-Wno-implicit-function-declaration
2256+
endif
2257+
endif
2258+
22182259
ifdef OVERRIDE_STRDUP
22192260
COMPAT_CFLAGS += -DOVERRIDE_STRDUP
22202261
COMPAT_OBJS += compat/strdup.o

compat/.gitattributes

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1 +1,2 @@
11
/zlib-uncompress2.c whitespace=-indent-with-non-tab,-trailing-space
2+
/mimalloc/**/* whitespace=-trailing-space

compat/posix.h

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -176,6 +176,16 @@ typedef unsigned long uintptr_t;
176176
#define _ALL_SOURCE 1
177177
#endif
178178

179+
#ifdef USE_MIMALLOC
180+
#include "mimalloc.h"
181+
#define malloc mi_malloc
182+
#define calloc mi_calloc
183+
#define realloc mi_realloc
184+
#define free mi_free
185+
#define strdup mi_strdup
186+
#define strndup mi_strndup
187+
#endif
188+
179189
#ifdef MKDIR_WO_TRAILING_SLASH
180190
#define mkdir(a,b) compat_mkdir_wo_trailing_slash((a),(b))
181191
int compat_mkdir_wo_trailing_slash(const char*, mode_t);

config.mak.dev

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -22,8 +22,10 @@ endif
2222

2323
ifneq ($(uname_S),FreeBSD)
2424
ifneq ($(or $(filter gcc6,$(COMPILER_FEATURES)),$(filter clang7,$(COMPILER_FEATURES))),)
25+
ifndef USE_MIMALLOC
2526
DEVELOPER_CFLAGS += -std=gnu99
2627
endif
28+
endif
2729
else
2830
# FreeBSD cannot limit to C99 because its system headers unconditionally
2931
# rely on C11 features.

config.mak.uname

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -521,7 +521,7 @@ endif
521521
CC = compat/vcbuild/scripts/clink.pl
522522
AR = compat/vcbuild/scripts/lib.pl
523523
CFLAGS =
524-
BASIC_CFLAGS = -nologo -I. -Icompat/vcbuild/include -DWIN32 -D_CONSOLE -DHAVE_STRING_H -D_CRT_SECURE_NO_WARNINGS -D_CRT_NONSTDC_NO_DEPRECATE
524+
BASIC_CFLAGS = -nologo -I. -Icompat/vcbuild/include -DWIN32 -D_CONSOLE -DHAVE_STRING_H -D_CRT_SECURE_NO_WARNINGS -D_CRT_NONSTDC_NO_DEPRECATE -MP -std:c11
525525
COMPAT_OBJS = compat/msvc.o compat/winansi.o \
526526
compat/win32/flush.o \
527527
compat/win32/path-utils.o \

0 commit comments

Comments
 (0)