Skip to content

Commit d480f9f

Browse files
fix(vllm-cpp): restore backend build portability
Keep the current master pin when enabling MLX so every backend variant builds against the known-good vllm.cpp revision. Suppress Apple clang’s GNU constant-folding diagnostic for Objective-C++ Metal compilation only, since upstream treats warnings as errors. Assisted-by: Codex:gpt-5 [systematic-debugging]
1 parent cc67378 commit d480f9f

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

backend/go/vllm-cpp/Makefile

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ JOBS?=$(shell nproc --ignore=1 2>/dev/null || sysctl -n hw.ncpu 2>/dev/null || e
1111

1212
# vllm.cpp version
1313
VLLM_CPP_REPO?=https://github.com/mudler/vllm.cpp
14-
VLLM_CPP_VERSION?=59343930f37ba08abfccb8de544bc51fc3828b16
14+
VLLM_CPP_VERSION?=9e1c9025ae61167a3335454d7cc0de6093c21845
1515

1616
# MLX GEMM provider (darwin/metal only; see the metal branch below for why).
1717
# Consumed as the prebuilt pip wheel: building MLX from source needs `xcrun
@@ -71,7 +71,7 @@ ifeq ($(BUILD_TYPE),cublas)
7171
else ifeq ($(BUILD_TYPE),vulkan)
7272
CMAKE_ARGS+=-DVLLM_CPP_VULKAN=ON -DVLLM_CPP_CUDA=OFF
7373
else ifeq ($(BUILD_TYPE),metal)
74-
CMAKE_ARGS+=-DVLLM_CPP_METAL=ON
74+
CMAKE_ARGS+=-DVLLM_CPP_METAL=ON -DCMAKE_OBJCXX_FLAGS=-Wno-gnu-folding-constant
7575
# The optional MLX GEMM provider. vllm.cpp keeps it OFF by default because it
7676
# is a ~19 MB libmlx.dylib plus a ~105 MB mlx.metallib, and upstream's
7777
# position is that it must earn that cost by measurement. It does, on the

0 commit comments

Comments
 (0)