You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
main (current head, via commit b53e20f85e "Implement ROCm split-KV
attention decode (issue #845)", authored 2026-09-17) fails to build for gfx1200 under the project's -Werror HIP arm. Two unused-variable
diagnostics in src/vt/rocm/rocm_paged_attn.hip:
Discovered while independently reproducing the operator's gfx1200 toggle
matrix for PR #3036 (KERNEL-QUANT-CIQ-GEMM-ROCM's cooperative-tile WMMA
arms). That PR's own changed files are unaffected — git diff --stat between
the row branch's pre-merge tip and its current head (after merging main)
touches only src/vt/rocm/rocm_grouped_gemm.hip and tests/vt/test_backend_cross_device.cpp. rocm_paged_attn.hip is untouched
by #3036; the break is inherited purely from the main merge.
git blame on both lines attributes them to b53e20f85e alone.
Impact
Any current gfx1200 build (this row's toggle matrix included) fails before a
single device test can run. Whatever CI runs for this repo does not build
ROCm/HIP (no such check appears on recent PRs, #3036 included), so this
never surfaced there — only a real gfx1200 box building current main
reproduces it.
No row claims this yet. Per AGENTS.md "An issue you do not fix in the same
flow has to say who owns it," this needs either an owning row's Row: line
or a spec's ## Owed entry — filed as - because neither exists yet; needs
triage to pick an owner (most likely folded into BACKEND-ROCM or a fresh
row for the gfx1200 -Werror arm).
Activity