Skip to content

fix: symbol collision for ggml_vec_dot_q4_1_q8_1 when building for wasm#24639

Merged
ggerganov merged 1 commit into
ggml-org:masterfrom
abetlen:fix/wasm-fallback-symbol
Jun 15, 2026
Merged

fix: symbol collision for ggml_vec_dot_q4_1_q8_1 when building for wasm#24639
ggerganov merged 1 commit into
ggml-org:masterfrom
abetlen:fix/wasm-fallback-symbol

Conversation

@abetlen

@abetlen abetlen commented Jun 15, 2026

Copy link
Copy Markdown
Collaborator

Overview

cont #22209

ggml_vec_dot_q4_1_q8_1_generic is renamed here for wasm builds even though ggml_vec_dot_q4_1_q8_1 was recently implemented in wasm/quants.c, this breaks builds with CMAKE_SYSTEM_PROCESSOR=wasm.

Requirements

@abetlen abetlen requested a review from ggerganov as a code owner June 15, 2026 06:42
@github-actions github-actions Bot added the ggml changes relating to the ggml tensor library for machine learning label Jun 15, 2026
@ggerganov ggerganov added the merge ready A maintainer can use this label to indicate that they consider the changes final and ready to merge. label Jun 15, 2026
@ggerganov ggerganov merged commit 6eab471 into ggml-org:master Jun 15, 2026
24 of 27 checks passed
am17an pushed a commit to am17an/llama.cpp that referenced this pull request Jun 20, 2026
papamoose pushed a commit to papamoose/llama.cpp that referenced this pull request Jun 27, 2026
adrianhoehne pushed a commit to adrianhoehne/llama.cpp that referenced this pull request Jul 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ggml changes relating to the ggml tensor library for machine learning merge ready A maintainer can use this label to indicate that they consider the changes final and ready to merge.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants