deps: point ggml to Adreno fused qknorm rope#4
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: e9986d486b
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| @@ -1 +1 @@ | |||
| Subproject commit 399ea717940d2008e70ead58a992b83d0745569f | |||
| Subproject commit 765887922710717e15892b6d5207a22eef74abd3 | |||
There was a problem hiding this comment.
Point ggml at a fetchable commit
This updates the ggml gitlink to 7658879, but .gitmodules still fetches the submodule from https://github.com/leejet/ggml.git; that repository does not contain this commit while it exists in the happyyzy/ggml fork referenced by the commit message. In a fresh checkout, git submodule update --init ggml will fail unless the user already has that object locally, so builds from this revision cannot reliably fetch dependencies. Please either push/merge the ggml commit to the configured submodule remote or update the submodule URL to a remote that contains it.
Useful? React with 👍 / 👎.
Summary
Updates the
ggmlsubmodule pointer to the local staging branch with the fused Adreno qknorm/RoPE OpenCL path.Corresponding ggml PR: happyyzy/ggml#1.
Notes
This PR is stacked on #3 and only updates the dependency pointer.