Skip to content

Commit 1fe3480

Browse files
committed
GH-45978: [C++] Bump bundled mimalloc version
Update bundled mimalloc version from 2.0.6 to 2.2.4. This will also help fix the CMake 4.0 compatibility issues.
1 parent d2a8b78 commit 1fe3480

2 files changed

Lines changed: 11 additions & 7 deletions

File tree

cpp/cmake_modules/ThirdpartyToolchain.cmake

Lines changed: 9 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -2275,22 +2275,26 @@ if(ARROW_MIMALLOC)
22752275
# We only use a vendored mimalloc as we want to control its build options.
22762276

22772277
set(MIMALLOC_LIB_BASE_NAME "mimalloc")
2278-
if(WIN32)
2279-
set(MIMALLOC_LIB_BASE_NAME "${MIMALLOC_LIB_BASE_NAME}-static")
2280-
endif()
22812278
if(${UPPERCASE_BUILD_TYPE} STREQUAL "DEBUG")
22822279
set(MIMALLOC_LIB_BASE_NAME "${MIMALLOC_LIB_BASE_NAME}-${LOWERCASE_BUILD_TYPE}")
22832280
endif()
22842281

22852282
set(MIMALLOC_PREFIX "${CMAKE_CURRENT_BINARY_DIR}/mimalloc_ep/src/mimalloc_ep")
2286-
set(MIMALLOC_INCLUDE_DIR "${MIMALLOC_PREFIX}/include/mimalloc-2.0")
2283+
set(MIMALLOC_INCLUDE_DIR "${MIMALLOC_PREFIX}/include/mimalloc-2.2")
22872284
set(MIMALLOC_STATIC_LIB
2288-
"${MIMALLOC_PREFIX}/lib/mimalloc-2.0/${CMAKE_STATIC_LIBRARY_PREFIX}${MIMALLOC_LIB_BASE_NAME}${CMAKE_STATIC_LIBRARY_SUFFIX}"
2285+
"${MIMALLOC_PREFIX}/lib/mimalloc-2.2/${CMAKE_STATIC_LIBRARY_PREFIX}${MIMALLOC_LIB_BASE_NAME}${CMAKE_STATIC_LIBRARY_SUFFIX}"
22892286
)
22902287

2288+
set(MIMALLOC_C_FLAGS)
2289+
if(WIN32)
2290+
# Workaround https://github.com/microsoft/mimalloc/issues/910 on RTools40
2291+
set(MIMALLOC_C_FLAGS "${MIMALLOC_C_FLAGS} -DERROR_COMMITMENT_MINIMUM=635")
2292+
endif()
2293+
22912294
set(MIMALLOC_CMAKE_ARGS
22922295
${EP_COMMON_CMAKE_ARGS}
22932296
"-DCMAKE_INSTALL_PREFIX=${MIMALLOC_PREFIX}"
2297+
"-DCMAKE_C_FLAGS=${MIMALLOC_C_FLAGS}"
22942298
-DMI_OVERRIDE=OFF
22952299
-DMI_LOCAL_DYNAMIC_TLS=ON
22962300
-DMI_BUILD_OBJECT=OFF

cpp/thirdparty/versions.txt

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -82,8 +82,8 @@ ARROW_JEMALLOC_BUILD_VERSION=5.3.0
8282
ARROW_JEMALLOC_BUILD_SHA256_CHECKSUM=2db82d1e7119df3e71b7640219b6dfe84789bc0537983c3b7ac4f7189aecfeaa
8383
ARROW_LZ4_BUILD_VERSION=v1.10.0
8484
ARROW_LZ4_BUILD_SHA256_CHECKSUM=537512904744b35e232912055ccf8ec66d768639ff3abe5788d90d792ec5f48b
85-
ARROW_MIMALLOC_BUILD_VERSION=v2.0.6
86-
ARROW_MIMALLOC_BUILD_SHA256_CHECKSUM=9f05c94cc2b017ed13698834ac2a3567b6339a8bde27640df5a1581d49d05ce5
85+
ARROW_MIMALLOC_BUILD_VERSION=v2.2.4
86+
ARROW_MIMALLOC_BUILD_SHA256_CHECKSUM=754a98de5e2912fddbeaf24830f982b4540992f1bab4a0a8796ee118e0752bda
8787
ARROW_NLOHMANN_JSON_BUILD_VERSION=v3.12.0
8888
ARROW_NLOHMANN_JSON_BUILD_SHA256_CHECKSUM=4b92eb0c06d10683f7447ce9406cb97cd4b453be18d7279320f7b2f025c10187
8989
ARROW_OPENTELEMETRY_BUILD_VERSION=v1.21.0

0 commit comments

Comments
 (0)