Skip to content

Add extension subgroup-size-control#5578

Open
Jiawei-Shao wants to merge 23 commits into
gpuweb:mainfrom
Jiawei-Shao:add-subgroup-size-control
Open

Add extension subgroup-size-control#5578
Jiawei-Shao wants to merge 23 commits into
gpuweb:mainfrom
Jiawei-Shao:add-subgroup-size-control

Conversation

@Jiawei-Shao
Copy link
Copy Markdown
Contributor

@Jiawei-Shao Jiawei-Shao commented Feb 27, 2026

This patch adds a new WebGPU and WGSL extension subgroup-size-control
based on subgroup-size-control.md.

Fixed: #5545

@Jiawei-Shao Jiawei-Shao marked this pull request as draft February 27, 2026 07:19
@Jiawei-Shao Jiawei-Shao marked this pull request as ready for review February 27, 2026 07:58
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Feb 27, 2026

Previews, as seen when this build job started (a963977):
WebGPU webgpu.idl | Explainer | Correspondence Reference
WGSL grammar.js | wgsl.lalr.txt

@dneto0 dneto0 requested review from dneto0 February 27, 2026 14:40
Comment thread spec/index.bs Outdated
Comment thread spec/index.bs Outdated
Comment thread spec/index.bs Outdated
Comment thread wgsl/index.bs Outdated
Comment thread wgsl/index.bs
Comment thread wgsl/index.bs Outdated
@Jiawei-Shao Jiawei-Shao marked this pull request as draft March 4, 2026 07:59
@Jiawei-Shao Jiawei-Shao marked this pull request as ready for review March 4, 2026 08:02
@jimblandy
Copy link
Copy Markdown
Contributor

Should this PR also update the correspondence reference?

@Jiawei-Shao Jiawei-Shao marked this pull request as draft March 12, 2026 03:06
@Jiawei-Shao Jiawei-Shao marked this pull request as ready for review March 12, 2026 03:17
@Jiawei-Shao
Copy link
Copy Markdown
Contributor Author

Should this PR also update the correspondence reference?

Done

@Jiawei-Shao
Copy link
Copy Markdown
Contributor Author

PTAL, thanks!

Comment thread spec/index.bs Outdated
Comment thread spec/index.bs Outdated
Comment thread spec/index.bs Outdated
Comment thread spec/index.bs Outdated
Comment thread spec/index.bs Outdated
Comment thread spec/index.bs Outdated
Comment thread spec/index.bs Outdated
Copy link
Copy Markdown
Contributor

@jimblandy jimblandy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

My comments above were mostly editorial, but there was one spot that required a fix.

Copy link
Copy Markdown

@mwyrzykowski mwyrzykowski left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In general the concept seems fine as an optional extension, as noted on Metal this property is read only and can not be configured as far as I know.

@Jiawei-Shao
Copy link
Copy Markdown
Contributor Author

PTAL, thanks!

Comment thread spec/index.bs Outdated
@Jiawei-Shao Jiawei-Shao requested a review from jrprice April 17, 2026 02:35
@Jiawei-Shao
Copy link
Copy Markdown
Contributor Author

Hi @jimblandy and @mwyrzykowski ,

I've updated this PR. PTAL, thanks!

Comment thread spec/index.bs Outdated
Comment thread spec/index.bs Outdated
Comment thread spec/index.bs Outdated
Comment thread spec/index.bs
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should have CTS and prototype for this before the spec lands, so we can validate the design before committing to it.

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.

Subgroup Size Control

6 participants