Skip to content
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.

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

ggml : fix BLAS with unsupported types #9775

Merged
merged 2 commits into from
Oct 8, 2024
Merged

Conversation

slaren
Copy link
Collaborator

@slaren slaren commented Oct 7, 2024

  • Do not use BLAS with types that don't have to_float functions
  • Change ggml_internal_get_type_traits to return a const pointer to avoid copies in sensitive paths
  • Rename ggml_internal_get_type_traits to ggml_get_type_traits

Fixes #9766

slaren added 2 commits October 7, 2024 22:04
ggml : return pointer from ggml_internal_get_type_traits to avoid unnecessary copies
it's not really internal if everybody uses it
@github-actions github-actions bot added testing Everything test related Vulkan Issues specific to the Vulkan backend examples ggml changes relating to the ggml tensor library for machine learning labels Oct 7, 2024
@slaren slaren merged commit dca1d4b into master Oct 8, 2024
54 checks passed
@slaren slaren deleted the sl/fix-blas-and-traits branch October 8, 2024 12:21
dsx1986 pushed a commit to dsx1986/llama.cpp that referenced this pull request Oct 29, 2024
* ggml : do not use BLAS with types without to_float

* ggml : return pointer from ggml_internal_get_type_traits to avoid unnecessary copies

* ggml : rename ggml_internal_get_type_traits -> ggml_get_type_traits

it's not really internal if everybody uses it
arthw pushed a commit to arthw/llama.cpp that referenced this pull request Nov 15, 2024
* ggml : do not use BLAS with types without to_float

* ggml : return pointer from ggml_internal_get_type_traits to avoid unnecessary copies

* ggml : rename ggml_internal_get_type_traits -> ggml_get_type_traits

it's not really internal if everybody uses it
arthw pushed a commit to arthw/llama.cpp that referenced this pull request Nov 18, 2024
* ggml : do not use BLAS with types without to_float

* ggml : return pointer from ggml_internal_get_type_traits to avoid unnecessary copies

* ggml : rename ggml_internal_get_type_traits -> ggml_get_type_traits

it's not really internal if everybody uses it
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
examples ggml changes relating to the ggml tensor library for machine learning testing Everything test related Vulkan Issues specific to the Vulkan backend
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bug: BLAS causes Segmentation Fault
2 participants