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

[Backport to 19] add reverse translation for OpDecorateString and OpMemberDecorateString #2767

Merged

Conversation

svenvh
Copy link
Member

@svenvh svenvh commented Oct 18, 2024

No description provided.

…emberDecorateString (KhronosGroup#2677)

This PR adds "reverse translation" (from SPIR-V to LLVM IR) support for OpDecorateString and OpMemberDecorateString, see KhronosGroup#2460 and KhronosGroup#2670. These instructions are currently treated as synonyms for OpDecorate and OpMemberDecorate. We'll want to tidy this up at some point, but at least for now we won't crash if we see these instructions.

I'll still need to add proper support for decorating variables in the input storage class for KhronosGroup#2670, but I'll do that in a separate PR.

(cherry picked from commit f7057b4)
@svenvh svenvh changed the title Backport 19 decoratestring [Backport to 19] add reverse translation for OpDecorateString and OpMemberDecorateString Oct 18, 2024
@svenvh svenvh merged commit 8764661 into KhronosGroup:llvm_release_190 Oct 18, 2024
9 checks passed
@svenvh svenvh deleted the backport-19-decoratestring branch October 18, 2024 14:45
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.

2 participants