Upgrade llama.cpp from b9555 to b9621#225
Conversation
No project source changes required — all upstream API breaks in this range (ggml_gated_delta_net state-tensor reshape, common_get_device_memory_data return-type change, mtmd_helper_bitmap_* return-type change, llm_graph_result::set_outputs signature change) are absorbed inside upstream-compiled translation units. New upstream features in this range (EAGLE3 speculative decoding, video input pipeline, mtmd_batch_max_tokens, path_prompts_log_dir, ggml_col2im_1d op) are noted in the breaking-changes doc as candidates for future Java API exposure. https://claude.ai/code/session_016jPq9MLePa3eXjxiLLStwi
|
Review Summary: Upgrade llama.cpp from b9555 to b9621 - APPROVED. All three required files correctly updated (CMakeLists.txt, README.md, CLAUDE.md). 15 upstream changes documented comprehensively in breaking-changes history with zero project source impact. All changes verified as internal to backend TUs or upstream-compiled translation units. Analysis quality is excellent with new features noted for future Java API exposure. Low risk, well-documented bump. |
TODO.md, docs/feature-investigation-similar-projects.md, and docs/history/llama-cpp-breaking-changes.md were missing SPDX-FileCopyrightText / SPDX-License-Identifier tags, causing the REUSE compliance check to report 3 non-compliant files (211/214). https://claude.ai/code/session_016jPq9MLePa3eXjxiLLStwi
|
|
Review Summary: This PR successfully upgrades llama.cpp from b9555 to b9621. APPROVED - All three required locations updated consistently: CMakeLists.txt, README.md, and CLAUDE.md. Comprehensive breaking changes documentation added (15 entries) correctly identifies that all API changes are internal to llama.cpp backends or consumed only by upstream-compiled translation units. Zero project source impact as documented. SPDX license headers added appropriately. Follows established CLAUDE.md upgrade procedure exactly. No security concerns. Well-executed, properly documented upstream version bump. Merge as-is. |



Summary
server-context.cpp,speculative.cpp,mtmd-helper.cpp)Changes
Configuration updates:
CMakeLists.txt: UpdatedGIT_TAGfrom b9555 to b9621README.md: Updated badge and link to reflect b9621CLAUDE.md: Updated pinned version reference to b9621Breaking changes documentation:
Added 15 entries to
docs/history/llama-cpp-breaking-changes.mdcovering:ggml_gated_delta_netstate tensor reshape (Qwen3.5/Qwen3-Next internal kernel)ggml_col2im_1doperationcommon_get_device_memory_data()return type change (consumed in upstreamserver-context.cpp)mtmd_helper_bitmap_init_*()return type changes (verified zero project call sites)mtmd_helper_video_*,--videoCLI flag)common_paramsfields (path_prompts_log_dir,mtmd_batch_max_tokens)llama-ext.hllm_graph_result::set_outputs()signature changeVK_VALVE_shader_mixed_float_dot_product)Test plan
Related issues / PRs
Follows the llama.cpp upgrade procedure documented in CLAUDE.md.
Checklist
CONTRIBUTING.mdandCODE_OF_CONDUCT.mdhttps://claude.ai/code/session_016jPq9MLePa3eXjxiLLStwi