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

Add type checks for hitObjectNV #3689

Merged
merged 3 commits into from
Aug 8, 2024
Merged

Add type checks for hitObjectNV #3689

merged 3 commits into from
Aug 8, 2024

Conversation

D7ry
Copy link
Contributor

@D7ry D7ry commented Aug 7, 2024

VK_NV_ray_tracing_invocation_reorder extension introduces hitObjectNV, a special opaque type
that can only be declared without storage qualifiers in either global or function scope.
Added checks/tests to enforce this constraint.

References:
https://registry.khronos.org/vulkan/specs/1.3-extensions/man/html/VK_NV_ray_tracing_invocation_reorder.html
https://github.com/KhronosGroup/GLSL/blob/3e0d9a3b3f54651ef53d533392a7401a3b19e16d/extensions/nv/GLSL_NV_shader_invocation_reorder.txt#L180

D7ryNV and others added 2 commits August 2, 2024 11:31
`VK_NV_ray_tracing_invocation_reorder` extension introduces `hitObjectNV`, a special opaque type
that can only be declared without storage qualifiers in either global or function scope.
Added checks/tests to enforce this constraint.

References:
https://registry.khronos.org/vulkan/specs/1.3-extensions/man/html/VK_NV_ray_tracing_invocation_reorder.html
https://github.com/KhronosGroup/GLSL/blob/3e0d9a3b3f54651ef53d533392a7401a3b19e16d/extensions/nv/GLSL_NV_shader_invocation_reorder.txt#L180
@CLAassistant
Copy link

CLAassistant commented Aug 7, 2024

CLA assistant check
All committers have signed the CLA.

@arcady-lunarg arcady-lunarg added the kokoro:run Trigger Google bot runs label Aug 8, 2024
@kokoro-team kokoro-team removed the kokoro:run Trigger Google bot runs label Aug 8, 2024
Copy link
Contributor

@arcady-lunarg arcady-lunarg left a comment

Choose a reason for hiding this comment

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

Overall I think this is a good change, but I would prefer a slight rewording of the error message in the struct case.

glslang/MachineIndependent/ParseHelper.cpp Outdated Show resolved Hide resolved
@arcady-lunarg arcady-lunarg added the kokoro:run Trigger Google bot runs label Aug 8, 2024
Copy link
Contributor

@arcady-lunarg arcady-lunarg 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.

@kokoro-team kokoro-team removed the kokoro:run Trigger Google bot runs label Aug 8, 2024
@arcady-lunarg arcady-lunarg merged commit 7c4d91e into KhronosGroup:main Aug 8, 2024
29 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants