Skip to content

webgpu : reorder fattn includes to fix unresolved V error - #27545

Merged
fairydreaming merged 1 commit into
ggml-org:masterfrom
fairydreaming:webgpu-fattn-v-unresolved
Aug 24, 2026
Merged

webgpu : reorder fattn includes to fix unresolved V error#27545
fairydreaming merged 1 commit into
ggml-org:masterfrom
fairydreaming:webgpu-fattn-v-unresolved

Conversation

@fairydreaming

@fairydreaming fairydreaming commented Aug 22, 2026

Copy link
Copy Markdown
Contributor

Overview

If KV_OVERLAP is defined flash_attn_decls.tmpl defines V as K, but after #25956 first occurrence of V is now in common_decls.tmpl that is included earlier. This causes Error while parsing WGSL: :40:16 error: unresolved value 'V' errors - probably in recently added V-is-view-of-K FA tests from #27394. This PR reorders these includes to ensure V macro is defined before first V occurrence.

Requirements

…ay be defined as K in flash_attn_decls.tmpl if KV_OVERLAP
@fairydreaming
fairydreaming requested a review from a team as a code owner August 22, 2026 14:25
@github-actions github-actions Bot added ggml changes relating to the ggml tensor library for machine learning WebGPU labels Aug 22, 2026
@fairydreaming
fairydreaming merged commit c1c766d into ggml-org:master Aug 24, 2026
20 of 24 checks passed
therealkenc pushed a commit to therealkenc/llama.cpp that referenced this pull request Aug 24, 2026
…ay be defined as K in flash_attn_decls.tmpl if KV_OVERLAP (ggml-org#27545)

Co-authored-by: Stanisław Szymczyk <sszymczy@gmail.com>
gagallo7 pushed a commit to gagallo7/qvac-fabric-llm.cpp that referenced this pull request Aug 27, 2026
…ay be defined as K in flash_attn_decls.tmpl if KV_OVERLAP (ggml-org#27545)

Co-authored-by: Stanisław Szymczyk <sszymczy@gmail.com>
(cherry picked from commit c1c766d)
thecodacus pushed a commit to thecodacus/llama.cpp that referenced this pull request Sep 7, 2026
…ay be defined as K in flash_attn_decls.tmpl if KV_OVERLAP (ggml-org#27545)

Co-authored-by: Stanisław Szymczyk <sszymczy@gmail.com>
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 WebGPU

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants