oneMKL icon indicating copy to clipboard operation
oneMKL copied to clipboard

Don't use `sycl::detail` namespace

Open AlexeySachkov opened this issue 1 year ago • 2 comments

sycl::detail::pi is used in:

https://github.com/oneapi-src/oneMKL/blob/6c5f7ea783a7fe828c0de3c064c3bc837727524d/src/blas/backends/cublas/cublas_scope_handle.cpp#L123

Please note there detail sub-namespace is reserved for implementations and there is no guarantees about its content. Not only that is non-portable, but it is also unmaintainable, because there is always risk that internal implementation details will change tomorrow.

AlexeySachkov avatar Oct 23 '23 08:10 AlexeySachkov

@AlexeySachkov Thank you for reporting this. Do you have a suggestion for how to accomplish what we are doing here in some other way?

andrewtbarker avatar Nov 22 '23 17:11 andrewtbarker

Hi @andrewtbarker, unfortunately I don't see any core or an extension-provided feature which would expose the same functionality. Therefore, it seems to me that you may need to request a corresponding extension to be added to have this API formalized - this won't be portable still, but at least it would not be relying on implementation details.

AlexeySachkov avatar Nov 23 '23 11:11 AlexeySachkov