Skip to content

Conversation

@VyacheslavLevytskyy
Copy link
Contributor

We introduce here SPIR-V Backend support in Clang/Driver

@VyacheslavLevytskyy
Copy link
Contributor Author

It was agreed that we split #16317 into three parts:

@asudarsa
Copy link
Contributor

Hi @VyacheslavLevytskyy

The changes are small and look fine to me. However, the PR topic is a bit misleading. We have already introduced SPIR-V backend support and this change just updates the target triple. Can you please reword it accordingly? Or let me know if I might be missing something here.

Thanks

@VyacheslavLevytskyy VyacheslavLevytskyy changed the title [Clang] Introduce SPIR-V Backend support in Clang/Driver [Clang] Update of invoking SPIR-V Backend in Clang/Driver Jan 13, 2025
@VyacheslavLevytskyy
Copy link
Contributor Author

Hi @VyacheslavLevytskyy

The changes are small and look fine to me. However, the PR topic is a bit misleading. We have already introduced SPIR-V backend support and this change just updates the target triple. Can you please reword it accordingly? Or let me know if I might be missing something here.

Thanks

Thank you, you are absolutely right. I changed the title.

@againull againull merged commit 3b39297 into intel:sycl Jan 13, 2025
18 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