[Piglit] [PATCH] Modify the FindOpenCL.make to add the CL lib name
junyan.he at linux.intel.com
Thu May 16 00:58:50 PDT 2013
On 05/16/2013 01:44 AM, Tom Stellard wrote:
> On Wed, May 15, 2013 at 02:57:53PM +0800, junyan.he at linux.intel.com wrote:
>> From: Junyan He <junyan.he at linux.intel.com>
>> Some OpenCL lib's name is not libOpenCL.so.
>> Some thing like libCL.so, libcl.so may exist,
>> and in these cases the check failed.
>> Add the names to fix this problem.
>> Signed-off-by: Junyan He <junyan.he at linux.intel.com>
>> cmake/Modules/FindOpenCL.cmake | 3 ++-
>> 1 file changed, 2 insertions(+), 1 deletion(-)
>> diff --git a/cmake/Modules/FindOpenCL.cmake b/cmake/Modules/FindOpenCL.cmake
>> index cc66714..121dcf2 100644
>> --- a/cmake/Modules/FindOpenCL.cmake
>> +++ b/cmake/Modules/FindOpenCL.cmake
>> @@ -35,7 +35,8 @@
>> find_path(OPENCL_INCLUDE_PATH CL/opencl.h)
>> - find_library(OPENCL_opencl_LIBRARY OpenCL)
>> + find_library(OPENCL_opencl_LIBRARY
>> + NAMES OpenCL CL cl)
> Which OpenCL implementations create libCL.so or libcl.so ? Is the
> purpose of this to test implementations without going through the ICD
yes, the intel's openCL open source code implement called beignet use
the name libcl.so.
And on my platform I do not want to install ICD every time, so I want to
loader my CL lib
directly. I think it will have no harm to the current behavior, because
it will check the
existence of libOpenCL.so firstly.
> I don't know much about CMake, but this patch is OK with me as long as
> it doesn't break the current behavior.
>> Piglit mailing list
>> Piglit at lists.freedesktop.org
More information about the Piglit