HIP icon indicating copy to clipboard operation
HIP copied to clipboard

not all "pi" functions present in ROCm math library are available through hipcc

Open VinInn opened this issue 2 years ago • 4 comments

ROCm math library implements all "pi" functions: sinpi, cospi, tanpi and their inverse (including atan2pi) hip seems to provide access only to sinpi and cospi.

see https://github.com/RadeonOpenCompute/ROCm-Device-Libs/issues/99

VinInn avatar Sep 21 '23 14:09 VinInn

Will raise it internally. Open it up to broader discussion since if we add it, it will be supported for a long time.

cjatin avatar Sep 21 '23 14:09 cjatin

It seems we should not document that tanpi is supported.

tanpi is not declared by CUDA SDK https://docs.nvidia.com/cuda/cuda-c-programming-guide/index.html#mathematical-functions-appendix .

yxsamliu avatar Sep 21 '23 15:09 yxsamliu

@VinInn Is this ticket still relevant? Thanks!

ppanchad-amd avatar Apr 29 '24 15:04 ppanchad-amd

It would be useful to have all math functions implemented in the device library available through hipcc. I do understand that at the moment strict compatibility with the CUDA SDK is instead preferred.

VinInn avatar Apr 29 '24 15:04 VinInn