Skip to content

hexagon: add SGN op - #30346

Merged
max-krasnyansky merged 5 commits into
ggml-org:masterfrom
qualcomm:kurquhar/hexagon-sgn
Oct 23, 2026
Merged

max-krasnyansky merged 5 commits into
ggml-org:masterfrom
qualcomm:kurquhar/hexagon-sgn

Conversation

@kurquhar

@kurquhar kurquhar commented Oct 11, 2026 •

Copy link
Copy Markdown
Contributor

Overview

Add SGN op to hexagon backend; motivated by Qwen3.8-Flash-Next (qwen4exp)

Additional information

New SGN op added. Only FP32 supported at this time, but could be extended to FP16, if needed.

Spills and other checks are clean in the new kernels.

Verification via test-backend-ops:

  • 4 existing F32 SGN cases changed from unsupported to passing
  • 8 F16/BF16 SGN cases remain unsupported by design

Requirements

  • I have read and agree with the contributing guidelines Y
  • AI usage disclosure: Yes, reasoning, authoring, and test execution.

Assisted-by: OpenCode

@kurquhar

Copy link
Copy Markdown
Contributor Author

@max-krasnyansky for visibility

@github-actions github-actions Bot added ggml changes relating to the ggml tensor library for machine learning Hexagon labels Oct 11, 2026
@kurquhar
kurquhar force-pushed the kurquhar/hexagon-sgn branch from 1fa4ad4 to ec99af2 Compare October 23, 2026 01:01
@kurquhar
kurquhar marked this pull request as ready for review October 23, 2026 01:02
@kurquhar
kurquhar requested a review from a team as a code owner October 23, 2026 01:02
Comment thread ggml/src/ggml-hexagon/htp/unary-ops.c Outdated
@max-krasnyansky

Copy link
Copy Markdown
Member

@kurquhar let's rebase with the latest fusion changes and remove that check (see comment in the code)

@kurquhar
kurquhar force-pushed the kurquhar/hexagon-sgn branch from ec99af2 to dac32f8 Compare October 23, 2026 13:57
@kurquhar

Copy link
Copy Markdown
Contributor Author

@max-krasnyansky rebased and scalar fallback removed

@max-krasnyansky

Copy link
Copy Markdown
Member

@lhez for another review/ack

@max-krasnyansky max-krasnyansky added the merge ready A maintainer can use this label to indicate that they consider the changes final and ready to merge. label Oct 23, 2026
@max-krasnyansky
max-krasnyansky merged commit 0ae0e19 into ggml-org:master Oct 23, 2026
25 checks passed
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 Hexagon 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.

3 participants