search

clMathLibraries / clBLAS

a software library containing BLAS functions written in OpenCL
Apache License 2.0
839 stars 240 forks source link

Always cast get_*_* to uints before casting to vectors. #65

Closed pavanky closed 9 years ago

pavanky commented 9 years ago
TimmyLiu commented 9 years ago

Hi Pavan,

Thanks for fixing this bug. Can you verify one thing for me that c/zgemm are not broken. I remember a while back some one in issue #53 was trying to fix a similar bug but it broke cgemm. Can you just run a full short test on your system with "./test-short --gtest_filter=gemm"?

pavanky commented 9 years ago

I ran test-short for GEMM for AMD GPUs on Linux. I have also tested cgemm / zgemm indirectly on other devices using ArrayFire. All of them seem to be passing at this point.

Do you want me to explicitly test on anything else ?

TimmyLiu commented 9 years ago

thanks! Just being precautions. I merged the pull request.