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

[SPIR-V] enable [[vk::ext_decorate/_id/_string]] for struct/class fields or function parameter #4195

Closed
jaebaek opened this issue Jan 18, 2022 · 0 comments · Fixed by #7137
Closed
Assignees
Labels
spirv Work related to SPIR-V

Comments

@jaebaek
Copy link
Collaborator

jaebaek commented Jan 18, 2022

As a part of HLSL version of GL_EXT_spirv_intrinsics (#3919), we have to enable [[vk::ext_decorate/_id/_string]] for struct/class fields or function parameter.

@jaebaek jaebaek added the spirv Work related to SPIR-V label Jan 18, 2022
@llvm-beanz llvm-beanz moved this to For Google in HLSL Triage Jul 24, 2024
@s-perron s-perron added this to the Next Release milestone Aug 22, 2024
@s-perron s-perron moved this from For Google to Triaged in HLSL Triage Aug 22, 2024
@s-perron s-perron self-assigned this Feb 11, 2025
@s-perron s-perron moved this from New to In progress in HLSL Roadmap Feb 11, 2025
s-perron added a commit to s-perron/DirectXShaderCompiler that referenced this issue Feb 11, 2025
This commit implements adding the `vk::ext_decorate` on a field. It also
adds an error in sema if `vk::ext_decorate_id` or
`vk::ext_decorate_string` are used on members.

For `vk::ext_decorate_id`, there is no `OpMemberDecorateId` instruction,
so we cannot add apply these decorations to members.

For `vk::ext_decorate_string`, there is only one decoration that uses
OpDecorateString or OpMemberDecorateString, UserSemantic. However, that
decoration could have used OpDecorate or OpMemberDecorate because those
instructions accept string literals. I do not expect any new decorations
to use OpDecorateString or OpMemberDecorateString. I may eventually want
to deprecate `vk::ext_decorate_string`.

Fixes microsoft#4195
@s-perron s-perron moved this from In progress to In review in HLSL Roadmap Feb 11, 2025
@github-project-automation github-project-automation bot moved this from In review to Done in HLSL Roadmap Feb 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
spirv Work related to SPIR-V
Projects
Status: Done
Status: Triaged
Development

Successfully merging a pull request may close this issue.

2 participants