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

SYCLomatic README missing a pre-requisite and a Useful link #1296

Open
mpanoop opened this issue Sep 12, 2023 · 2 comments
Open

SYCLomatic README missing a pre-requisite and a Useful link #1296

mpanoop opened this issue Sep 12, 2023 · 2 comments
Labels
bug Something isn't working

Comments

@mpanoop
Copy link
Contributor

mpanoop commented Sep 12, 2023

The pre-requisite section of the README document for SYCLomatic repo doesn't call out the CUDA headers dependency and versions supported (https://github.com/oneapi-src/SYCLomatic#prerequisites)

The post migration SYCL can be targeted for non-Intel devices as well but the useful links doesn't point to right articles which explains how to do this. Below are suggested links to link to:

  1. Targeting nVIDIA GPUs - https://developer.codeplay.com/products/oneapi/nvidia/2023.2.1/guides/get-started-guide-nvidia#use-dpc-to-target-nvidia-gpus
  2. Targeting AMD GPUs - https://developer.codeplay.com/products/oneapi/amd/2023.2.1/guides/get-started-guide-amd#use-dpc-to-target-amd-gpus
@mpanoop mpanoop added the bug Something isn't working label Sep 12, 2023
@mpanoop
Copy link
Contributor Author

mpanoop commented Sep 12, 2023

Building SYCLomatic from source doesn't need CUDA headers but using SYCLomatic to migrate the CUDA application to SYCL needs CUDA headers. Can we call this out explicitly in README?

@tomflinda
Copy link
Contributor

@mpanoop thanks for your advice, we will plan to update the README file.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants