Skip to content
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

Descriptor type validation doesn't handle OpGroupDecorate #20

Closed
karl-lunarg opened this issue May 14, 2018 · 6 comments
Closed

Descriptor type validation doesn't handle OpGroupDecorate #20

karl-lunarg opened this issue May 14, 2018 · 6 comments
Assignees
Labels
Bug Something isn't working
Milestone

Comments

@karl-lunarg
Copy link
Contributor

Issue by phaulos (MIGRATED)
Thursday Mar 09, 2017 at 22:22 GMT
Originally opened as KhronosGroup/Vulkan-LoaderAndValidationLayers#1549


dEQP-VK.spirv_assembly.instruction.compute.decoration_group.all is failing with:

ERROR: Type mismatch on descriptor slot 0.0 (used as type `ptr to uniform struct of (oddtype)`) but descriptor of type VK_DESCRIPTOR_TYPE_STORAGE_BUFFER (code 0x0000000d from SC at Unknown:2631)
...

Looking at the SPIR-V BufferBlock decoration is applied to 0.0 through OpGroupDecorate. This is not something glslang does so it might be that validation simply doesn't handle it.

@karl-lunarg karl-lunarg added this to the P1 milestone May 14, 2018
@karl-lunarg karl-lunarg added the Bug Something isn't working label May 14, 2018
@karl-lunarg
Copy link
Contributor Author

Comment by chrisforbes (MIGRATED)
Friday Mar 10, 2017 at 03:58 GMT


Indeed, SC has no idea about group decorations at all. This is going to take a bit of work.

@karl-lunarg
Copy link
Contributor Author

Comment by chrisforbes (MIGRATED)
Friday Mar 24, 2017 at 04:53 GMT


Working out with spirv-tools side how best to deal with this.

@karl-lunarg
Copy link
Contributor Author

Comment by gwihlidal (MIGRATED)
Wednesday May 17, 2017 at 00:10 GMT


Any update on this? We're hitting this error in many of our shaders.

@karl-lunarg
Copy link
Contributor Author

Comment by mark-lunarg (MIGRATED)
Thursday Mar 01, 2018 at 21:40 GMT


@chrisforbes, any update? Still applicable?

@mark-lunarg
Copy link
Contributor

@chrisforbes, is this still an issue?

@chrisforbes
Copy link
Contributor

Yes, still needed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants