Update llama.cpp patch queue - #1014
Conversation
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (17)
📝 WalkthroughWalkthroughThe patch series extends llama.cpp with Skippy staged execution, activation sidebands, session and KV state transfer, resident-prefix caching, runtime events, device enumeration, native MTP execution, chat-template reasoning formats, and chat auto-parser coverage. ChangesSkippy staged runtime
Estimated code review effort: 5 (Critical) | ~120 minutes Possibly related PRs
Suggested reviewers: ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
This pull request is currently a draft. Reviews will not take place until the PR is marked as ready for review. |
Summary
99f3dc32296f825fec94f202da1e9fede1e78cf9toe8f19cc0ad70a243c8012bf17b4be601abfc8ea20015to0015/0016Why
The existing patch queue no longer applied cleanly to current llama.cpp master. Patch 2 conflicted in
src/models/minimax-m2.cppafter upstream added layer-input bookkeeping.Impact
Keeps the branch-local staged llama.cpp runtime current while preserving the existing Skippy ABI and patch-series semantics. The range comparison shows the MiniMax M2 conflict resolution is the only semantic adaptation; the remaining carried commits are unchanged.
Validation
scripts/prepare-llama.sh pinnedjust buildcargo test -p mesh-llm --lib(41 passed)Summary by CodeRabbit