Skip to content

ggml : fix field name when new ggml_backend #14944

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 1 commit into from
Aug 8, 2025
Merged

ggml : fix field name when new ggml_backend #14944

merged 1 commit into from
Aug 8, 2025

Conversation

aisk
Copy link
Contributor

@aisk aisk commented Jul 29, 2025

The field name is actually iface:

llama.cpp/ggml/src/ggml-backend-impl.h

Line 121 in 138b288

struct ggml_backend_i iface;

@aisk aisk requested a review from 0cc4m as a code owner July 29, 2025 13:19
@github-actions github-actions bot added Nvidia GPU Issues specific to Nvidia GPUs Vulkan Issues specific to the Vulkan backend ggml changes relating to the ggml tensor library for machine learning SYCL https://en.wikipedia.org/wiki/SYCL - GPU programming language OpenCL Issues specific to the OpenCL backend labels Jul 29, 2025
@slaren slaren merged commit cd6983d into ggml-org:master Aug 8, 2025
47 checks passed
@abrimogard abrimogard mentioned this pull request Aug 8, 2025
Closed
@aisk aisk deleted the fix-field-name branch August 8, 2025 14:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@slaren slaren slaren approved these changes

@0cc4m 0cc4m Awaiting requested review from 0cc4m 0cc4m is a code owner

Assignees
No one assigned
Labels
ggml changes relating to the ggml tensor library for machine learning Nvidia GPU Issues specific to Nvidia GPUs OpenCL Issues specific to the OpenCL backend SYCL https://en.wikipedia.org/wiki/SYCL - GPU programming language Vulkan Issues specific to the Vulkan backend
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@aisk @slaren