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

Fix SPIRV error for WorkGraphs #6423

Conversation

jkwak-work
Copy link
Collaborator

This commit resolves an error message that is caused while emitting SPIRV code.

The cause was on the fact that there is no corresponding IRInst for one of SPIRV instructions when emitting,

%in_payload_t = OpTypeNodePayloadArrayAMDX $$T;
%in_payload_ptr_t = OpTypePointer NodePayloadAMDX %in_payload_t;

There is an IRInst for OpTypePointer but there isn't IRInst corresponding to OpTypeNodePayloadArrayAMDX. When Slang emits OpTypePointer with its second operand OpTypeNodePayloadArrayAMDX, it is going to be emitted without a matching IRInst.

Closes #6391

This commit resolves an error message that is caused while emitting
SPIRV code.

The cause was on the fact that there is no corresponding IRInst for
one of SPIRV instructions when emitting,
```
%in_payload_t = OpTypeNodePayloadArrayAMDX $$T;
%in_payload_ptr_t = OpTypePointer NodePayloadAMDX %in_payload_t;
```

There is an IRInst for `OpTypePointer` but there isn't IRInst
corresponding to `OpTypeNodePayloadArrayAMDX`. When Slang emits
`OpTypePointer` with its second operand `OpTypeNodePayloadArrayAMDX`,
it is going to be emitted without a matching IRInst.
@jkwak-work jkwak-work added the pr: non-breaking PRs without breaking changes label Feb 22, 2025
@jkwak-work jkwak-work self-assigned this Feb 22, 2025
@jkwak-work jkwak-work requested a review from a team as a code owner February 22, 2025 01:05
@jkwak-work
Copy link
Collaborator Author

Closing it because I realized that this is not a good way to solve this.

@jkwak-work jkwak-work closed this Feb 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr: non-breaking PRs without breaking changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[WorkGraphs] experimental shader code causes Slang to crash
1 participant