SYCLomatic icon indicating copy to clipboard operation
SYCLomatic copied to clipboard

SYCLomatic README missing a pre-requisite and a Useful link

Open mpanoop opened this issue 1 year ago • 2 comments

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 avatar Sep 12 '23 20:09 mpanoop

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?

mpanoop avatar Sep 12 '23 21:09 mpanoop

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

tomflinda avatar Sep 13 '23 05:09 tomflinda

@mpanoop we have added the note: Note: SYCLomatic can be built from source without any CUDA dependencies. However, before migration of CUDA codebases to SYCL, ensure that CUDA header files are accessible to the tool. These header files are necessary for SYCLomatic to properly understand and process CUDA code during the migration process. in the https://github.com/oneapi-src/SYCLomatic#prerequisites

tomflinda avatar Jun 05 '24 02:06 tomflinda