Upgrade llama.cpp from b9842 to b9859#290
Merged
Merged
Conversation
Bumps the pinned llama.cpp tag in llama/CMakeLists.txt (GIT_TAG + -DLLAMA_TAG), the README badge/link, and the CLAUDE.md version references. The b9842..b9859 range touches no header the project binds; it is almost entirely upstream backend TUs (CPU nvfp4 vec_dot, CUDA fattn head-dim-512 + get_rows_back grid clamp, OpenCL Q1_0 + Adreno binary kernels, Vulkan Asahi, WebGPU NVFP4, Hexagon FA rework) plus internal common/arg.cpp (hf-split primary) and common/http.h (IPv6 host bracketing) changes — all compiled inside upstream TUs. No jllama.cpp change required. All four patches re-verified to apply cleanly against b9859 via git apply --check over the fetched b9859 sources: the only changed patch-target file is common/arg.cpp, whose b9859 edit (common_models_handler_apply) is disjoint from patch 0001's make_utf8_argv/common_params_parse hunks; 0002/0003/0004 target files untouched in this range. OuteTTS generator anchors hold (tools/tts/tts.cpp unchanged). Appends the upgrade rows to docs/history/llama-cpp-breaking-changes.md. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_019pnrHT4MP55QV8MWTo2pf1
|
✅ Code review complete. Summary: This PR upgrades llama.cpp from b9842 to b9859 with thorough, consistent updates across all configuration and documentation files. Findings:
Verdict: Ready to merge. The upgrade is low-risk and well-documented. |
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.



Summary
Test plan
Related issues / PRs
N/A
Checklist
CONTRIBUTING.mdandCODE_OF_CONDUCT.mdhttps://claude.ai/code/session_019pnrHT4MP55QV8MWTo2pf1