[Piglit] [PATCH] cl: -cl-std=CL1.0 is not valid

Tom Stellard tom at stellard.net
Thu Dec 11 18:10:42 PST 2014


On Wed, Dec 03, 2014 at 11:04:50PM +0100, EdB wrote:
> By OpenCL 1.2 spec, -cl-std= should only be CL1.1 or CL1.2
> We were checking against the device version instead of the desired version
> 
> Report by: yan.wang at linux.intel.com

I've pushed this, thanks!

-Tom

> ---
>  tests/util/piglit-framework-cl-program.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/tests/util/piglit-framework-cl-program.c b/tests/util/piglit-framework-cl-program.c
> index 374296f..d335486 100644
> --- a/tests/util/piglit-framework-cl-program.c
> +++ b/tests/util/piglit-framework-cl-program.c
> @@ -218,7 +218,8 @@ piglit_cl_program_test_run(const int argc,
>  		sprintf(build_options+strlen(old), "%s", config->build_options);
>  		free(old);
>  	}
> -	if(version > 10) {
> +
> +	if(env.clc_version > 10) {
>  		//If -cl-std was already in config->build_options, use what the test requested
>  		if (!strstr(build_options, "-cl-std")){
>  			char* template = " -cl-std=CL%d.%d";
> -- 
> 2.2.0
> 
> _______________________________________________
> Piglit mailing list
> Piglit at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/piglit


More information about the Piglit mailing list