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 14] Support both legacy and Khronos version of global variable extensions (#2228) #2557

Closed

Conversation

wenju-he
Copy link
Contributor

@wenju-he wenju-he commented May 9, 2024

This is done to provide smooth drivers transition to official extensions.
Also updated tests for new extensions to use new tokens after KhronosGroup/SPIRV-Headers@cca08c6

(cherry picked from commit 32786ff)

…riable extensions (KhronosGroup#2228)

This is done to provide smooth drivers transition to official
extensions.
Also updated tests for new extensions to use new tokens after
KhronosGroup/SPIRV-Headers@cca08c6

(cherry picked from commit 32786ff)
@wenju-he
Copy link
Contributor Author

wenju-he commented May 9, 2024

There is an igc test which uses SPV_INTEL_global_variable_decorations fails after #2495. I'll update the test to change SPV_INTEL_global_variable_decorations to SPV_INTEL_global_variable_host_access.
Not sure if #2495 would impact more tests, so I backport this PR.
@vmaksimo could you please check if this backport makes sense? If yes, I'll backport to 15/16/17 branches as well.

@wenju-he
Copy link
Contributor Author

wenju-he commented May 10, 2024

a dozen of new regressions are reported on #2495

@MrSidims MrSidims requested a review from vmaksimo May 15, 2024 22:36
@wenju-he wenju-he closed this May 20, 2024
@wenju-he wenju-he deleted the cherry-pick-2228 branch May 20, 2024 23:52
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.

None yet

2 participants