Skip to content

Conversation

@jmmartinez
Copy link

This change should not change any tests since the SPIRV tests are currently not used.

@jmmartinez jmmartinez requested a review from jplehr December 15, 2025 14:22
@jmmartinez jmmartinez self-assigned this Dec 15, 2025
@jplehr jplehr requested review from Kewen12 and Meinersbur December 15, 2025 14:37
@jplehr
Copy link
Contributor

jplehr commented Dec 15, 2025

My thoughts are that we want to spin-up a new builder that would cover the SPIRV tests and leave this one untouched.

@jmmartinez
Copy link
Author

My thoughts are that we want to spin-up a new builder that would cover the SPIRV tests and leave this one untouched.

But in that case, how are we going to run the simple tests? Through hip-tpl.py ?

@jplehr
Copy link
Contributor

jplehr commented Dec 15, 2025

In the long run I would like to transition towards a more configurable Python implementation, like hip-tpl.py. But I can see that we want to go this route first.

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.

3 participants