Skip to content

Adjusts subgroup built-in name verbs to match atomics operations#4627

Merged
kdashg merged 1 commit intogpuweb:mainfrom
munrocket:subgroup-ops
Jun 4, 2024
Merged

Adjusts subgroup built-in name verbs to match atomics operations#4627
kdashg merged 1 commit intogpuweb:mainfrom
munrocket:subgroup-ops

Conversation

@munrocket
Copy link
Copy Markdown
Contributor

This PR is part of #4528 which only changes two operations

subgroupSum -> subgroupAdd
subgroupProduct -> subgroupMul

And making builtin's consistent with GLSL, WGSL atomics and wgpu.

This PR is part of gpuweb#4528 which only changes two operations
```
subgroupSum -> subgroupAdd
subgroupProduct -> subgroupMul
```
And making builtin's consistent with GLSL, WGSL atomics and wgpu.
@w3cbot
Copy link
Copy Markdown

w3cbot commented May 8, 2024

munrocket marked as non substantive for IPR from ash-nazg.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented May 8, 2024

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

@kainino0x kainino0x added the wgsl WebGPU Shading Language Issues label May 21, 2024
@kdashg kdashg changed the title Adjusts subgroups built-in to atomics operations Adjusts subgroup built-in name verbs to match atomics operations Jun 4, 2024
@mwyrzykowski mwyrzykowski self-requested a review June 4, 2024 18:13
@kdashg
Copy link
Copy Markdown
Contributor

kdashg commented Jun 4, 2024

WGSL 2024-06-04 Minutes
  • AB: We don’t have a strong opinion. Two languages did it “sum” and “product” and one language did “add” and “mul”. HLSL uses InterlockedAdd.
  • RESOLVED: ACCEPTED

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

wgsl WebGPU Shading Language Issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants