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

android: Update SPIRV-Tools #7050

Closed
wants to merge 1 commit into from
Closed

android: Update SPIRV-Tools #7050

wants to merge 1 commit into from

Conversation

juan-lunarg
Copy link
Contributor

SPIRV-Tools no longer links against android libraries it doesn't use. See KhronosGroup/SPIRV-Tools#5482

Now VVL must explicitly link against android

@juan-lunarg juan-lunarg requested a review from a team as a code owner November 29, 2023 21:05
@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build queued with queue ID 92035.

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build queued with queue ID 92044.

SPIRV-Tools no longer links against android libraries it doesn't
use. See KhronosGroup/SPIRV-Tools#5482

Now VVL must explicitly link against `android`
@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build queued with queue ID 92056.

juan-lunarg referenced this pull request in KhronosGroup/SPIRV-Tools Nov 29, 2023
* cmake: Simplify usage of option

boolean OFF is the default value:
https://cmake.org/cmake/help/latest/command/option.html

* Fix iOS / Android CMake builds

closes #4437
@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ValidationLayers build # 14701 running.

@juan-lunarg
Copy link
Contributor Author

Seeing some new test failures. Out of scope for what I originally intended this PR to be. Just keep in mind this Android change for when we do update SPIRV-Tools.

@juan-lunarg juan-lunarg deleted the juan/update_spirv_tools branch November 29, 2023 21:53
jeremyg-lunarg added a commit to jeremyg-lunarg/Vulkan-ValidationLayers that referenced this pull request Dec 4, 2023
Remove stage specific debug info that is only needed by GPU-AV.
This allows debug printfs to be used in multi stage shader modules.

Requires KhronosGroup/SPIRV-Tools#5495

Fixes KhronosGroup#4350

Also includes the VUL linking change from KhronosGroup#7050
jeremyg-lunarg added a commit that referenced this pull request Dec 5, 2023
Remove stage specific debug info that is only needed by GPU-AV.
This allows debug printfs to be used in multi stage shader modules.

Requires KhronosGroup/SPIRV-Tools#5495

Fixes #4350

Also includes the VUL linking change from #7050
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.

2 participants