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

EdB edb+piglit at sigluy.net
Wed Dec 3 14:04:50 PST 2014


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
---
 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



More information about the Piglit mailing list