Skip to content

hexagon: add FP32 GELU_ERF and GEGLU_ERF support - #29631

Merged
max-krasnyansky merged 3 commits into
ggml-org:masterfrom
qualcomm:aparmp/hexagon-gelu-erf
Sep 29, 2026
Merged

max-krasnyansky merged 3 commits into
ggml-org:masterfrom
qualcomm:aparmp/hexagon-gelu-erf

Conversation

@aparmp-quic

Copy link
Copy Markdown
Contributor

Overview

Add FP32 Hexagon HTP support for GELU_ERF and GEGLU_ERF. GELU_ERF is required by the Qwen3 0.6B ASR model's CLIP graph. GEGLU_ERF is added alongside it for completeness.

Additional information

The implementation uses an HVX vectorized erf approximation with boundary handling for numerical stability. Inputs outside [-10, 10] use GELU's saturated behavior to avoid problematic intermediate values.

Requirements

  • I have read and agree with the contributing guidelines
  • AI usage disclosure: YES - AI was used for implementation guidance, and documentation review. The changes were reviewed and validated manually by the contributor.

@aparmp-quic
aparmp-quic requested a review from a team as a code owner September 29, 2026 06:25
@github-actions github-actions Bot added ggml changes relating to the ggml tensor library for machine learning Hexagon labels Sep 29, 2026
@CISC

CISC commented Sep 29, 2026

Copy link
Copy Markdown
Member

Just retriggering CI.

@CISC CISC closed this Sep 29, 2026
@CISC CISC reopened this Sep 29, 2026
@CISC

CISC commented Sep 29, 2026

Copy link
Copy Markdown
Member

Ah, nvm, rebase please.

@max-krasnyansky
max-krasnyansky merged commit e904318 into ggml-org:master Sep 29, 2026
18 checks passed
edwardyoon pushed a commit to edwardyoon/focus-llama that referenced this pull request Oct 7, 2026
* hexagon: add FP32 GELU_ERF and GEGLU_ERF support

* hex-erf: reduce register pressure in kernels

(cherry picked from commit e904318)
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants