Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@toji
Copy link
Member

@toji toji commented Jul 10, 2024

Fixes #4377

Updates the plain color texture table to separate out read-only and write-only storage bindings. bgra8unorm is only given write-only access if the feature is enabled and never given read-only access. All other formats retain the same access as before.

Fixes #4377

Updates the plain color texture table to separate out read-only and
write-only storage bindings. bgra8unorm is only given write-only access
if the feature is enabled and never given read-only access. All other
formats retain the same access as before.
@toji toji requested review from jimblandy and kainino0x July 10, 2024 18:59
@github-actions
Copy link
Contributor

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

Copy link
Member

@teoxoy teoxoy left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@kainino0x kainino0x left a comment

Choose a reason for hiding this comment

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

LGTM after today's meeting

@kainino0x kainino0x merged commit dab35d2 into main Jul 18, 2024
@kainino0x kainino0x deleted the write-only-bgra8 branch July 18, 2024 00:07
@kainino0x kainino0x added the needs-cts-issue This change requires tests (or would need tests if accepted), but may not have a CTS issue filed yet label Jul 18, 2024
@teoxoy teoxoy restored the write-only-bgra8 branch July 22, 2024 13:51
@teoxoy teoxoy deleted the write-only-bgra8 branch July 22, 2024 13:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-cts-issue This change requires tests (or would need tests if accepted), but may not have a CTS issue filed yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

The bgra8unorm-storage feature should most likely only enable write-only access

4 participants