-
Notifications
You must be signed in to change notification settings - Fork 88
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Unittests test_clone and test_create_cl_kernel fail #15
Comments
For the first one, it doesnt look very serious, just an extra filename line. I'm not sure where that comes from. I grepped the repository for |
(the second one is trickier. Unfortunatley I dont have access to a Radeon; you will basically need to poke around, to find out how far the 123.0f is getting. It should be being set by the kernel, https://github.com/hughperkins/cuda-on-cl/blob/master/test/gtest/test_hostside_opencl_funcs.cpp#L34 This should be a relatively straightforward test, nothing too weird happening. The test is pretty much written in OpenCL, no CUDA involved, so should be fairly easy to poke around. clenqueuereadbuffer is here: https://www.khronos.org/registry/OpenCL/sdk/1.0/docs/man/xhtml/clEnqueueReadBuffer.html Oh, it might be that you need to pass the queue into the |
exact llvm version: sys-devel/llvm 3.9.1-r1
(+ + means option is enabled and curently installed, - - is the oposite) I'm trying to get the same tests running on beignet, I can't get opencl working with it, nothing to do with cuda-on-cl. |
Please use llvm 3.8 |
test_hostside_opencl_funcs fixed, in c9d684c. Piccie on radeon and hd: |
(and for the other one, its a trivial issue you can ignore; but if you really want to fix it, the current supported fix is: install llvm-3.8 :-) ) |
Hello,
I ran the unit test and 2 of them fail:
test_struct_cloner.test_clone
test_hostside_opencl_funcs.test_create_cl_kernel
System info:
kernel: 4.10.10-gentoo
opencl: amdgpu-pro-opencl-16.60.1.379184 on top of amdgpu open source stack
llvm: 3.9.1
harware: AMD RX480
clinfo output:
How would you read these results ?
Thanks,
Thomas.
The text was updated successfully, but these errors were encountered: