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

[SPIR-V] Apply changes related to SPIR-V 1.4 #4886

Closed
wants to merge 6 commits into from

Conversation

vmaksimo
Copy link
Contributor

@vmaksimo vmaksimo commented Nov 3, 2021

Signed-off-by: Dmitry Sidorov dmitry.sidorov@intel.com

Original commit:
KhronosGroup/SPIRV-LLVM-Translator@c5b3c8e3283b

Signed-off-by: Dmitry Sidorov <dmitry.sidorov@intel.com>

Original commit:
KhronosGroup/SPIRV-LLVM-Translator@c5b3c8e3283b
@vmaksimo
Copy link
Contributor Author

vmaksimo commented Nov 3, 2021

/summary:run

@vmaksimo
Copy link
Contributor Author

vmaksimo commented Dec 8, 2021

/summary:run

KornevNikita and others added 2 commits December 28, 2021 16:59
This is a patch to expand the collection of entry point interfaces.
In SPIR-V 1.4 and later OpEntryPoint must list all global variables in the
interface. Also fix quoted string output in SPIRV text format.

Co-authored-by: Alexey Sotkin <alexey.sotkin@intel.com>

Original commit:
KhronosGroup/SPIRV-LLVM-Translator@352ea14
@vmaksimo vmaksimo changed the title Add SPIR-V 1.4 checks [SPIR-V] Apply changes related to SPIR-V 1.4 Dec 28, 2021
@vmaksimo
Copy link
Contributor Author

vmaksimo commented Mar 2, 2022

This PR enables patches mentioned in #3686

@github-actions github-actions bot added the Stale label Aug 30, 2022
@vmaksimo vmaksimo removed the Stale label Sep 2, 2022
@bader
Copy link
Contributor

bader commented Jan 13, 2023

@vmaksimo, should we close this? I see that Dmitry is already ready to commit the same change with #7493.

@vmaksimo
Copy link
Contributor Author

@bader Sure. Closing this one.

@vmaksimo vmaksimo closed this Jan 16, 2023
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.

4 participants