Skip to content

[SYCL][Doc] Update SYCL --offload-arch option. #15497

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

Closed

Conversation

srividya-sundaram
Copy link
Contributor

@srividya-sundaram srividya-sundaram commented Sep 24, 2024

Adopt LLVM Clang's --offload-arch option for SYCL offloading to Intel GPUs, Intel CPUs, NVidia and AMD GPUs.

@mdtoguchi mdtoguchi changed the title [Driver][Doc] Update SYCL --offload-arch option. [SYCL][Doc] Update SYCL --offload-arch option. Sep 24, 2024
@srividya-sundaram srividya-sundaram marked this pull request as ready for review September 24, 2024 17:28
@srividya-sundaram srividya-sundaram requested a review from a team as a code owner September 24, 2024 17:28
Copy link
Contributor

@maarquitos14 maarquitos14 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I see we only have lists with the accepted value for --offload-arch for Intel devices, should we do the same for third party devices?

@srividya-sundaram
Copy link
Contributor Author

srividya-sundaram commented Sep 24, 2024

I see we only have lists with the accepted value for --offload-arch for Intel devices, should we do the same for third party devices?

For SYCL offloading to Intel CPUs and GPUs, there is a mapping between the accepted values for --offload-arch and what is passed to the target-specific LLVM compiler (OCLOC or opencl-aot).
For NVidia GPUs and AMD GPUs, there is no such mapping. I can maybe add a list of accepted values for --offload-arch for NVidia and AMD GPUs.

@maarquitos14
Copy link
Contributor

For NVidia GPUs and AMD GPUs, there is no such mapping. I can maybe add a list of accepted values for --offload-arch for NVidia and AMD GPUs.

I think that would be helpful for users.

@mdtoguchi
Copy link
Contributor

@gmlueck, please take a look as you were part of the original conversation for these values.

@srividya-sundaram srividya-sundaram deleted the sycl-offload-arch-doc branch September 26, 2024 22:23
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