Skip to content

ci: remove manual llama.cpp Windows ARM64 build workflow#1194

Merged
Zack Li (zhiyuan8) merged 1 commit into
mainfrom
chore/remove-llama-cpp-windows-ci
Jul 21, 2026
Merged

ci: remove manual llama.cpp Windows ARM64 build workflow#1194
Zack Li (zhiyuan8) merged 1 commit into
mainfrom
chore/remove-llama-cpp-windows-ci

Conversation

@mengshengwu

@mengshengwu Mengsheng Wu (mengshengwu) commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Delete .github/workflows/build-llama-cpp-windows.yml, a workflow_dispatch-only workflow that built upstream ggml-org/llama.cpp on the windows-arm64 runner.
  • Its HTP .so signing (HEXAGON_HTP_CERT_PFX -> signtool / inf2cat / .cat) duplicated the release-path signing in .github/workflows/_build-sdk.yml:174-275, which stays as the single source of truth.
  • No other workflow or doc referenced it (checked .github/**, notes/**, docs/**, README, CONTRIBUTING).

Test plan

  • git grep -nE 'build-llama-cpp-windows|Build llama\.cpp \(Windows ARM64\)|llama-cpp-windows-arm64' -- ':!third-party' ':!.claude' returns no matches (excluding this PR's own diff).

Its HTP signing duplicates .github/workflows/_build-sdk.yml, which is
the release-path source of truth. The workflow was workflow_dispatch
only and unreferenced elsewhere.

Signed-off-by: Mengsheng Wu <mengshengwu@outlook.com>
@mengshengwu
Mengsheng Wu (mengshengwu) force-pushed the chore/remove-llama-cpp-windows-ci branch from ef9507e to f20c3cb Compare July 17, 2026 10:01
Maximilien Grzeczka (Maki-Grz) pushed a commit to Maki-Grz/GenieX that referenced this pull request Jul 19, 2026
geniex-bench reported the prefill token rate over the raw prompt
length, but the QAIRT engine pads input_ids up to the next 128-token
chunk before prefill. The reported tok/s therefore understated the
work the engine actually did.

For qairt cells, round prompt_tokens up to ceil(n/128)*128 and
recompute prefill_tps over the padded count. QAIRT's prompt_time equals
ttft, so rate stays consistent with prompt_tokens / prompt_time.
llama_cpp does no such padding and is left untouched.

Closes qualcomm#1194

Signed-off-by: RemiliaForever <remilia@koumakan.cc>
@zhiyuan8
Zack Li (zhiyuan8) merged commit 58c1af8 into main Jul 21, 2026
28 checks passed
@zhiyuan8
Zack Li (zhiyuan8) deleted the chore/remove-llama-cpp-windows-ci branch July 21, 2026 13:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants