<div dir="ltr"><div class="gmail_quote"><div dir="ltr">On Mon, Oct 22, 2018 at 6:16 PM Eric Anholt <<a href="mailto:eric@anholt.net">eric@anholt.net</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Jason Ekstrand <<a href="mailto:jason@jlekstrand.net" target="_blank">jason@jlekstrand.net</a>> writes:<br>
<br>
> diff --git a/src/mesa/drivers/dri/i965/brw_program.c b/src/mesa/drivers/dri/i965/brw_program.c<br>
> index f5ebd3c3b05..78050cda359 100644<br>
> --- a/src/mesa/drivers/dri/i965/brw_program.c<br>
> +++ b/src/mesa/drivers/dri/i965/brw_program.c<br>
> @@ -91,14 +91,14 @@ brw_create_nir(struct brw_context *brw,<br>
>  <br>
>        nir_remove_dead_variables(nir, nir_var_shader_in | nir_var_shader_out);<br>
>        nir_lower_returns(nir);<br>
> -      nir_validate_shader(nir);<br>
> +      nir_validate_shader(nir, NULL);<br>
>        NIR_PASS_V(nir, nir_lower_io_to_temporaries,<br>
>                   nir_shader_get_entrypoint(nir), true, false);<br>
>     } else {<br>
>        nir = prog_to_nir(prog, options);<br>
>        NIR_PASS_V(nir, nir_lower_regs_to_ssa); /* turn registers into SSA */<br>
>     }<br>
> -   nir_validate_shader(nir);<br>
> +   nir_validate_shader(nir, NULL);<br>
<br>
It seems like you ought to have valid where args here.  Other than that,<br>
patch 1-5 are:<br></blockquote><div><br></div><div>Yeah, I was lazy.  I added something.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Reviewed-by: Eric Anholt <<a href="mailto:eric@anholt.net" target="_blank">eric@anholt.net</a>><br></blockquote><div><br></div><div>Thanks! <br></div></div></div>