On 12/05/2017 06:10 AM, Timothy Arceri wrote: > Reviewed-by: Timothy Arceri <tarc...@itsqueeze.com> >
Thanks, Timothy. Any chance to review the remaining patches in the series? It would be nice to land this batch soon to focus on the actual linker stuff. cheers, Eduardo > On 04/12/17 20:21, Eduardo Lima Mitev wrote: >> This is the main fork of the shader compilation code-path, where a NIR >> shader is obtained by calling spirv_to_nir() or glsl_to_nir(), >> depending on its nature.. >> >> v2: Use 'spirv_data' member from gl_linked_shader to know which method >> to call. (Timothy Arceri) >> --- >> src/mesa/drivers/dri/i965/brw_program.c | 10 ++++++++-- >> 1 file changed, 8 insertions(+), 2 deletions(-) >> >> diff --git a/src/mesa/drivers/dri/i965/brw_program.c >> b/src/mesa/drivers/dri/i965/brw_program.c >> index 755d4973cc0..4043253a653 100644 >> --- a/src/mesa/drivers/dri/i965/brw_program.c >> +++ b/src/mesa/drivers/dri/i965/brw_program.c >> @@ -31,6 +31,7 @@ >> #include <pthread.h> >> #include "main/imports.h" >> +#include "main/glspirv.h" >> #include "program/prog_parameter.h" >> #include "program/prog_print.h" >> #include "program/prog_to_nir.h" >> @@ -73,9 +74,14 @@ brw_create_nir(struct brw_context *brw, >> ctx->Const.ShaderCompilerOptions[stage].NirOptions; >> nir_shader *nir; >> - /* First, lower the GLSL IR or Mesa IR to NIR */ >> + /* First, lower the GLSL/Mesa IR or SPIR-V to NIR */ >> if (shader_prog) { >> - nir = glsl_to_nir(shader_prog, stage, options); >> + if (shader_prog->_LinkedShaders[stage]->spirv_data) >> + nir = _mesa_spirv_to_nir(ctx, shader_prog, stage, options); >> + else >> + nir = glsl_to_nir(shader_prog, stage, options); >> + assert (nir); >> + >> nir_remove_dead_variables(nir, nir_var_shader_in | >> nir_var_shader_out); >> nir_lower_returns(nir); >> nir_validate_shader(nir); >> > _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev