Skip to content

Fix for issue #4197#4198

Merged
arcady-lunarg merged 1 commit intoKhronosGroup:mainfrom
ErikHogeman:fix_4197
Apr 28, 2026
Merged

Fix for issue #4197#4198
arcady-lunarg merged 1 commit intoKhronosGroup:mainfrom
ErikHogeman:fix_4197

Conversation

@ErikHogeman
Copy link
Copy Markdown
Contributor

Update access chain handling to be compatible with raw descriptor heap accesses.

@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Mar 24, 2026

CLA assistant check
All committers have signed the CLA.

@dneto0
Copy link
Copy Markdown
Contributor

dneto0 commented Apr 1, 2026

Please sign the CLA. We can't land your code without it.

And thank you.

@ErikHogeman
Copy link
Copy Markdown
Contributor Author

Please sign the CLA. We can't land your code without it.

And thank you.

Sorry, I missed that this was required, I have now signed it. Thanks for the ping.

Copy link
Copy Markdown
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.

@ErikHogeman You will need to add your new test to one of the lists in gtests/Spv.FromFile.cpp so that it actually gets run.

Change-Id: Icca07eb85ba5692675dfa6eec9fcd48cb38c0333
@ErikHogeman
Copy link
Copy Markdown
Contributor Author

@ErikHogeman You will need to add your new test to one of the lists in gtests/Spv.FromFile.cpp so that it actually gets run.

Thanks, not sure how I missed that. Fixed now.

@arcady-lunarg arcady-lunarg added the kokoro:run Trigger Google bot runs label Apr 28, 2026
@kokoro-team kokoro-team removed the kokoro:run Trigger Google bot runs label Apr 28, 2026
@arcady-lunarg arcady-lunarg merged commit 881a6fa into KhronosGroup:main Apr 28, 2026
27 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