hexagon: add alternative Hexagon NPU backend implementation - #27535
hexagon: add alternative Hexagon NPU backend implementation#27535zhouwg-jeffzhou wants to merge 1 commit into
Conversation
|
Hi @zhouwg-jeffzhou, thanks for your contribution! Per our contribution guidelines, the automated PR checker found the following issue(s) that need your attention:
Please note that maintainers reserve the right to make final decisions on PRs. If you believe there is a mistake, please comment below. |
|
@ggml-gh-bot, thanks for your kind reminder. I have already converted my opening PR to drat. |
5e69118 to
ead8b2d
Compare
|
@max-krasnyansky, sorry to bother you. I have carefully studied your feedback in #26373, thank you very much for providing many valuable suggestions. This new PR addresses your suggestions, and I would like to make some solid contributions to ggml‑hexagon. |
3b07df8 to
e5001b2
Compare
Assisted-by: Trae(GLM-5.3)
e5001b2 to
bb4ce7f
Compare
There was a problem hiding this comment.
after submitting the PR, I spent several hours reviewing source files and scripts on the GitHub page, trying to strictly align with Qualcomm's original behavior.
It sees that ggml/src/ggml‑hexagon/CMakeLists.txt and ggml/src/ggml‑hexagon/htp/CMakeLists.txt are already aligned with upstream.
I am not sure whether I should modify the backend name from Hexagon‑cDSP to HTP, so that the backend name is also strictly aligned with Qualcomm's original behavior.
Overview
Follow‑up to #26373
This PR adds FastRPC-based ggml-hexagon, an alternative backend for Qualcomm Hexagon NPU on Android, which can coexist with the existing official dspqueue-based ggml-hexagon implementation.
Additional information
details can be found at RFC-26227
Requirements