Skip to content

[ExecuTorch][WebGPU] Support general groups=1 Conv1d - #21665

Merged
JCNTH merged 2 commits into
gh/JCNTH/206/origfrom
gh/JCNTH/207/orig
Aug 7, 2026
Merged

JCNTH merged 2 commits into
gh/JCNTH/206/origfrom
gh/JCNTH/207/orig

Conversation

@pytorchbot

Copy link
Copy Markdown
Collaborator

This PR was created by the merge bot to help merge the original PR into the main branch.
ghstack PR number: #21600 by @JCNTH
^ Please use this as the source of truth for the PR details, comments, and reviews
ghstack PR base: https://github.com/pytorch/executorch/tree/gh/JCNTH/207/base
ghstack PR head: https://github.com/pytorch/executorch/tree/gh/JCNTH/207/head
Merge bot PR base: https://github.com/pytorch/executorch/tree/gh/JCNTH/206/orig
Merge bot PR head: https://github.com/pytorch/executorch/tree/gh/JCNTH/207/orig
Differential Revision: D114936145
@diff-train-skip-merge

**Add a buffer-only groups=1 Conv1d kernel for Voxtral encoder shapes**

The WebGPU backend handled only K=1 pointwise and depthwise Conv1d, leaving non-transposed groups=1 K=3 encoder convolutions unsupported.

**Before**

- Groups=1 K>1 graphs had no valid WebGPU route.

**After**

- A direct NCL buffer kernel supports groups=1 convolution with stride, padding, dilation, and optional bias.

Key changes:

- `add_conv1d_node` — validates fp32 operands, geometry, shapes, and shader index ranges.
- `conv1d.wgsl` — mirrors Vulkan output indexing in `backends/vulkan/runtime/graph/ops/glsl/conv1d.glsl:78`, stride/padding at `:86`, and the kernel FMA loop at `:108`.
- Dispatch and resize hooks — fold x/y workgroups and recompute live parameters safely.
- Generated sources — refresh WGSL headers, registry output, and drift digests.

The route remains limited to non-transposed groups=1 fp32 convolution; existing pointwise and depthwise routes are unchanged.

Co-authored-with: Claude Code.

Differential Revision: [D114936145](https://our.internmc.facebook.com/intern/diff/D114936145/)
ghstack-source-id: 411044636
Pull-Request: #21600
@pytorch-bot

pytorch-bot Bot commented Aug 7, 2026

Copy link
Copy Markdown

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/21665

Note: Links to docs will display an error until the docs builds have been completed.

This comment was automatically generated by Dr. CI and updates every 15 minutes.

@meta-cla meta-cla Bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Aug 7, 2026
@JCNTH
JCNTH merged commit f7045ea into gh/JCNTH/206/orig Aug 7, 2026
9 checks passed
@JCNTH
JCNTH deleted the gh/JCNTH/207/orig branch August 7, 2026 17:32

This branch was previously deployed

1 inactive deployment
cadence — 3fd9a877 Deployed Aug 7, 2026 by JCNTH
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants