If we are generating SPIR-V tests using already existing tests, we
would need both programs.

Note that we are not removing the option to configure where those
binaries are on the script using envvars, as that can be handy if you
want to use glslang from master compiled by yourself, and having
glslang from system installed.
---
 CMakeLists.txt | 17 +++++++++++++++++
 1 file changed, 17 insertions(+)

diff --git a/CMakeLists.txt b/CMakeLists.txt
index 435c40ea5..d5e9de3a9 100644
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -52,6 +52,23 @@ option(PIGLIT_BUILD_CL_TESTS "Build tests for OpenCL" OFF)
 option(PIGLIT_GENERATE_GL_SPIRV_TESTS
        "Generate ARB_gl_spirv tests by converting GLSL shader_tests to SPIR-V" 
OFF)
 
+if (PIGLIT_GENERATE_GL_SPIRV_TESTS)
+       find_program(GLSLANG_VALIDATOR_FOUND glslangValidator)
+       find_program(SPIRV_DIS_FOUND spirv-dis)
+
+       if(NOT GLSLANG_VALIDATOR_FOUND)
+               message(FATAL_ERROR "Failed fo find glslangValidator, needed to 
"
+                       "generate the ARB_gl_spirv generated tests."
+                       )
+       endif()
+
+       if(NOT SPIRV_DIS_FOUND)
+               message(FATAL_ERROR "Failed to find spirv-dis, needed "
+                       "to generate the ARB_gl_spirv generated tests."
+                       )
+               endif()
+endif()
+
 if(PIGLIT_BUILD_GL_TESTS)
        find_package(OpenGL REQUIRED)
 endif()
-- 
2.19.1

_______________________________________________
Piglit mailing list
Piglit@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/piglit

Reply via email to