Skip to content

Conversation

@slaren
Copy link
Member

@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

3 participants