Skip to content
This repository has been archived by the owner on Jan 13, 2025. It is now read-only.

HipSYCL to AdaptiveCpp update & fixes #493

Merged

Conversation

OuadiElfarouki
Copy link
Collaborator

@OuadiElfarouki OuadiElfarouki commented Jan 17, 2024

This patch updates the hipSYCL compilation in portBLAS including :

  • Necessary naming updates hipSYCL -> AdaptiveCpp)
  • Cmake changes to account for AdaptiveCpp compilation approaches
  • Relevant compatibility considerations of SYCL 2020 features and other extensions still not supported by AdaptiveCpp.
  • Updated documentation/readme.

Compilation flows / Device targets compiled successfully at this stage :

  • intel CPU : omp default flow (host), and generic SPIRV/OpenCL flow.
  • intel GPU : generic OpenCL/SPIRV flow, and generic SPIRV/LevelZero flow.
  • Nvidia GPU : generic PTX/cuda flow.
  • AMD GPU : generic HIP/amdgcn flow. (Tested with rocm 5.6.1)

Tests and benchmarks are conditionally disabled for some operators depending on the target device and AdaptiveCpp flow. Remaining tests (in each case) are passing for all tested devices above.

Copy link
Collaborator

@s-Nick s-Nick left a comment

Choose a reason for hiding this comment

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

Thank for your patch @OuadiElfarouki
I point out other hipsycl reference that could be updated to adaptiveCpp everything else LGTM

README.md Outdated Show resolved Hide resolved
include/operations/blas_constants.h Outdated Show resolved Hide resolved
cmake/Modules/SYCL.cmake Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
Copy link
Collaborator

@s-Nick s-Nick left a comment

Choose a reason for hiding this comment

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

A part that small suggestion, LGTM

@muhammad-tanvir-1211 muhammad-tanvir-1211 merged commit fec888f into codeplaysoftware:master Feb 29, 2024
3 checks passed
@s-Nick s-Nick mentioned this pull request Apr 22, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants