nlesc-recruit / cudawrappers

C++ wrapper for the Nvidia C libraries (e.g. CUDA driver, nvrtc, cuFFT etc.)
https://cudawrappers.readthedocs.io/en/latest/
Apache License 2.0
4 stars 5 forks source link

Add [[deprecated]] to cu::Context #292

Open csbnw opened 2 months ago

csbnw commented 2 months ago

Is your feature request related to a problem? Please describe. Now that cudawrappers supports HIP, cu::Context only properly works when using CUDA as a backend and are executed as no-ops with HIP.

Describe the solution you'd like We may want to discourage users from using these functions even with CUDA by adding [[deprecated]] tags.

Describe alternatives you've considered N.A.

Additional context Contexts are deprecated in HIP: https://rocm.docs.amd.com/projects/HIP/en/develop/doxygen/html/group___context.html

loostrum commented 2 months ago

Requires C++14, so blocked by #186