Skip to content

Conversation

jjerphan
Copy link

Hi!

Thank you for developing llama.cpp.

Updating the distribution of llama.cpp on conda-forge, I had to perform this change so that it could be built for Apple Metal on osx_arm64 (see conda-forge/llama.cpp-feedstock#62 which first has introduced this patch).

Signed-off-by: Julien Jerphanion <git@jjerphan.xyz>
@github-actions github-actions bot added ggml changes relating to the ggml tensor library for machine learning Apple Metal https://en.wikipedia.org/wiki/Metal_(API) labels Oct 13, 2025
@jjerphan jjerphan marked this pull request as ready for review October 13, 2025 17:54
@jjerphan jjerphan requested a review from ggerganov as a code owner October 13, 2025 17:54
@ggerganov
Copy link
Member

This is not a good solution - on my Mac the contents is always null. I'll push an alternative fix.

@ggerganov
Copy link
Member

Can you confirm that this works on your end: #16576

@jjerphan
Copy link
Author

Let's verify with: conda-forge/llama.cpp-feedstock#71.

@jjerphan
Copy link
Author

Its workflows are completing successfully (the failure of the linter's workflow is unrelated).

@ggerganov
Copy link
Member

Superseded by #16576

@ggerganov ggerganov closed this Oct 14, 2025
@jjerphan jjerphan deleted the apple-metal/change-gpuAdress-for-contents branch October 14, 2025 17:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Apple Metal https://en.wikipedia.org/wiki/Metal_(API) ggml changes relating to the ggml tensor library for machine learning

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants