-
Notifications
You must be signed in to change notification settings - Fork 909
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Disallow write-only storage buffers #4370
Comments
This is a bit of a gray issue. From the point of shader translation - it doesn't care. The lack of readable texture storages can be seen as a limitation from the pipeline interface side, and all those validations happen on wgpu side. |
See [`gpuweb/gpuweb`gfx-rs#4370](gpuweb/gpuweb#4370).
See [`gpuweb/gpuweb`gfx-rs#4370](gpuweb/gpuweb#4370).
See [`gpuweb/gpuweb`gfx-rs#4370](gpuweb/gpuweb#4370).
See [`gpuweb/gpuweb`gfx-rs#4370](gpuweb/gpuweb#4370).
See [`gpuweb/gpuweb`gfx-rs#4370](gpuweb/gpuweb#4370).
See [`gpuweb/gpuweb`gfx-rs#4370](gpuweb/gpuweb#4370).
See [`gpuweb/gpuweb`gfx-rs#4370](gpuweb/gpuweb#4370).
See [`gpuweb/gpuweb`gfx-rs#4370](gpuweb/gpuweb#4370).
See [`gpuweb/gpuweb`gfx-rs#4370](gpuweb/gpuweb#4370).
See [`gpuweb/gpuweb`gfx-rs#4370](gpuweb/gpuweb#4370).
See [`gpuweb/gpuweb`gfx-rs#4370](gpuweb/gpuweb#4370).
See [`gpuweb/gpuweb`gfx-rs#4370](gpuweb/gpuweb#4370).
See [`gpuweb/gpuweb`gfx-rs#4370](gpuweb/gpuweb#4370).
See [`gpuweb/gpuweb`gfx-rs#4370](gpuweb/gpuweb#4370).
See [`gpuweb/gpuweb`gfx-rs#4370](gpuweb/gpuweb#4370).
AFAICT, according to Resource Layout Compatibility, WGSL does not consider write-only storage buffers to be part of the resource interface.
Naga 0.8.5 appears to validate the following fragment of module-scope WGSL:
Behavior on
master
-as-of-this-post is similar. Inserting the following module-scope WGSL fragment intests/in/access.wgsl
results in a passingcargo test
:But trying to actually use such a binding through
wgpu
results in errors like:Because it's making the reasonable assumption that the shader wanted something readable.
The text was updated successfully, but these errors were encountered: