Re: [Piglit] [PATCH] cl: Add test for clCreateProgramWithBinary() v3

2014-10-23 Thread Francisco Jerez
Thanks, Reviewed-by: Francisco Jerez curroje...@riseup.net Tom Stellard thomas.stell...@amd.com writes: v2: - Fix filename in doxygen comments - Remove redundant binary status check. v3: - Add test cases for passing binary programs to clBuildProgram() and clCompileProgram().

[Piglit] [PATCH] cl: Add test for clCreateProgramWithBinary() v2

2014-10-21 Thread Tom Stellard
v2: - Fix filename in doxygen comments - Remove redundant binary status check. --- tests/cl.py | 1 + tests/cl/api/CMakeLists.cl.txt| 1 + tests/cl/api/create-program-with-binary.c | 177 ++ 3 files changed, 179

[Piglit] [PATCH] cl: Add test for clCreateProgramWithBinary() v3

2014-10-21 Thread Tom Stellard
v2: - Fix filename in doxygen comments - Remove redundant binary status check. v3: - Add test cases for passing binary programs to clBuildProgram() and clCompileProgram(). --- tests/cl.py | 1 + tests/cl/api/CMakeLists.cl.txt| 1 +

Re: [Piglit] [PATCH] cl: Add test for clCreateProgramWithBinary()

2014-10-15 Thread Francisco Jerez
Hi Tom, thanks for writing a test for this, two minor comments below. Tom Stellard thomas.stell...@amd.com writes: --- tests/cl.py | 1 + tests/cl/api/CMakeLists.cl.txt| 1 + tests/cl/api/create-program-with-binary.c | 189

[Piglit] [PATCH] cl: Add test for clCreateProgramWithBinary()

2014-10-14 Thread Tom Stellard
--- tests/cl.py | 1 + tests/cl/api/CMakeLists.cl.txt| 1 + tests/cl/api/create-program-with-binary.c | 189 ++ 3 files changed, 191 insertions(+) create mode 100644 tests/cl/api/create-program-with-binary.c diff --git