-
Notifications
You must be signed in to change notification settings - Fork 123
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
Raytracing: Shader-group-handle mapping #1690
Raytracing: Shader-group-handle mapping #1690
Conversation
- required for binding-table mapping/translation
CI gfxreconstruct build queued with queue ID 244873. |
CI gfxreconstruct build # 4705 running. |
CI gfxreconstruct build # 4705 failed. |
- copy-able, comparable, hash-able, no heap allocation required
7383e17
to
426be23
Compare
CI gfxreconstruct build queued with queue ID 244893. |
CI gfxreconstruct build # 4706 running. |
CI gfxreconstruct build # 4706 failed. |
CI gfxreconstruct build queued with queue ID 244965. |
CI gfxreconstruct build # 4709 running. |
CI gfxreconstruct build # 4709 passed. |
the PR belongs to this issue: #1526 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't see any issues with these changes.
thank you @nickdriscoll-lunarg! |
shader_group_handle_t
) that can be compared and hashed (-> can be used as key in std::unordered_map)decode::PipelineInfo
to map from capture- to replay-time handles.this matches @bradgrantham-lunarg's proof-of-concept, fleshing out some details, but please note:
no re-mapping is actually added/performed (that part of the proof-of-concept was still too hacky).
so this PR is only a necessary stepping-stone and should not change GFXR's behavior in any way.