Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly
Appearance settings

Add linear_indexing language feature#5554

Merged
alan-baker merged 6 commits intogpuweb:maingpuweb/gpuweb:mainfrom
alan-baker:linear-indexesalan-baker/gpuweb:linear-indexesCopy head branch name to clipboard
Mar 10, 2026
Merged

Add linear_indexing language feature#5554
alan-baker merged 6 commits intogpuweb:maingpuweb/gpuweb:mainfrom
alan-baker:linear-indexesalan-baker/gpuweb:linear-indexesCopy head branch name to clipboard

Conversation

@alan-baker
Copy link
Copy Markdown
Contributor

@alan-baker alan-baker commented Feb 8, 2026

  • Two new builtins:
    • global_invocation_index - linearized global_invocation_id
    • workgroup_index - linearized workgroup_id

Fixes #5154

alan-baker
alan-baker commented Feb 8, 2026
View reviewed changes
Comment thread spec/index.bs Outdated
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Feb 8, 2026 •
edited
Loading

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

mehmetoguzderin
mehmetoguzderin reviewed Feb 8, 2026
View reviewed changes
Copy link
Copy Markdown
Member

@mehmetoguzderin mehmetoguzderin left a comment

Choose a reason for hiding this comment

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

I had a brief look and a minor question came to mind: was linear_indexing or linear_indices also considered as a name, given the technical nature of the specification? Apologies if this was already raised during the call and I missed the decision.

@alan-baker
Copy link
Copy Markdown
Contributor Author

alan-baker commented Feb 8, 2026

I had a brief look and a minor question came to mind: was linear_indexing or linear_indices also considered as a name, given the technical nature of the specification? Apologies if this was already raised during the call and I missed the decision.

linear_indexing might be good. We didn't name on the call. I considered linear_indices, but wanted index specifically to match.

@mehmetoguzderin
Copy link
Copy Markdown
Member

mehmetoguzderin commented Feb 8, 2026

Understood, linear_indexing might provide a reasonable balance. Thank you.

@alan-baker
Add linear_indexing language feature
d5d2103 
* Two new builtins:
  * `global_invocation_index` - linearized `global_invocation_id`
  * `workgroup_index` - linearized `workgroup_id`
@alan-baker alan-baker changed the title Add linear_indexes language feature Add linear_indexing language feature Feb 8, 2026
jrprice
jrprice reviewed Feb 9, 2026
View reviewed changes
Comment thread spec/index.bs Outdated
Comment thread spec/index.bs Outdated
Comment thread wgsl/index.bs Outdated
@jimblandy
Copy link
Copy Markdown
Contributor

jimblandy commented Feb 9, 2026

I'll put this on the WGSL agenda once it comes out of draft.

@alan-baker
Copy link
Copy Markdown
Contributor Author

alan-baker commented Feb 16, 2026

Draft CTS: gpuweb/cts#4595

@alan-baker alan-baker marked this pull request as ready for review February 17, 2026 04:18
@alan-baker alan-baker mentioned this pull request Feb 23, 2026
Merged
8 tasks
dneto0
dneto0 approved these changes Feb 24, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@dneto0 dneto0 left a comment

Choose a reason for hiding this comment

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

approved with suggested wording tweaks, and optional notes.

Comment thread spec/index.bs Outdated
Comment thread spec/index.bs Outdated
Comment thread wgsl/index.bs
@alan-baker alan-baker requested a review from dneto0 February 26, 2026 19:42
dneto0
dneto0 reviewed Mar 3, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@dneto0 dneto0 left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@jimblandy
Copy link
Copy Markdown
Contributor

jimblandy commented Mar 10, 2026

Mozilla likes this change.

@jimblandy
Copy link
Copy Markdown
Contributor

jimblandy commented Mar 10, 2026

@mwyrzykowski Could you take a look at this on Apple's behalf?

@alan-baker alan-baker merged commit 1234113 into gpuweb:main Mar 10, 2026
4 checks passed
@alan-baker alan-baker deleted the linear-indexes branch March 10, 2026 21:41
@jimblandy jimblandy mentioned this pull request Mar 10, 2026
Closed
jenkins-chromium-github-sync Bot pushed a commit to armarok/chromium that referenced this pull request Mar 11, 2026
@alan-baker
WebGPU: add linear_indexing enum
a237aba 
Spec: gpuweb/gpuweb#5554
CTS: gpuweb/cts#4595

Both were approved and landed.


Bug: 491559358
Change-Id: I81585882201d0bf18b86d28e47be4125650683b0
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/7654497
Commit-Queue: Alan Baker <alanbaker@google.com>
Reviewed-by: Corentin Wallez <cwallez@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1597693}
@andyleiserson andyleiserson mentioned this pull request Mar 24, 2026
Open
alan-baker added a commit to alan-baker/webgpu-headers that referenced this pull request Mar 30, 2026
@alan-baker
Linear indexing language extension
a9baa4d 
Spec PR merged here:
gpuweb/gpuweb#5554
@alan-baker alan-baker mentioned this pull request Mar 30, 2026
Merged
Kangz pushed a commit to webgpu-native/webgpu-headers that referenced this pull request Mar 31, 2026
@alan-baker
Linear indexing language extension (#587)
dc16b3e 
Spec PR merged here:
gpuweb/gpuweb#5554
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jrprice jrprice jrprice left review comments

@dneto0 dneto0 dneto0 approved these changes

@mehmetoguzderin mehmetoguzderin mehmetoguzderin approved these changes

@mwyrzykowski mwyrzykowski mwyrzykowski approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

builtins with a 3D variant should all have 1D variants (global_invocation_index, workgroup_index) (WGSL proposed language change)

6 participants

@alan-baker @mehmetoguzderin @jimblandy @jrprice @dneto0 @mwyrzykowski
Morty Proxy This is a proxified and sanitized view of the page, visit original site.