intel / llvm

Intel staging area for llvm.org contribution. Home for Intel LLVM-based projects.
Other
1.25k stars 738 forks source link

[SYCL][CUDA] Recommended API for deprecated ones #3780

Closed mmeterel closed 3 years ago

mmeterel commented 3 years ago

oneMKL open source interfaces is using the context "get" API for CUBLAS backend. As noted below, it is being deprecated. What is the recommendation to replace this API?

https://github.com/intel/llvm/blob/bd581c5e04338dc068bdbedd8980ae4bf76bdbe1/sycl/include/CL/sycl/context.hpp#L191

bader commented 3 years ago

I think this message must be updated to include the replacement.

romanovvlad commented 3 years ago

@alexbatashev Could you please help?

alexbatashev commented 3 years ago

oneMKL open source interfaces is using the context "get" API for CUBLAS backend

That is not a correct usage of APIs. get() method does not return a native handle for CUDA backend, and instead returns this structure.

What is the recommendation to replace this API?

You can use get_native

mmeterel commented 3 years ago

@alexbatashev Thanks for the recommendation. It works. Closing.