<p dir="ltr"><br>
On Apr 10, 2015 1:42 PM, "Rob Clark" <<a href="mailto:robdclark@gmail.com">robdclark@gmail.com</a>> wrote:<br>
><br>
> From: Rob Clark <<a href="mailto:robclark@freedesktop.org">robclark@freedesktop.org</a>><br>
><br>
> I guess I was looking too much at how lower_system_values worked when<br>
> writing lower_idiv.<br>
><br>
> Since ttn wasn't emitting load_var for sysvals and the only drivers<br>
> using lower_idiv were using ttn, I think nothing was broken as a result.<br>
> But might as well fix this before it becomes a problem.</p>
<p dir="ltr">Yup. No harm done.</p>
<p dir="ltr">Reviewed-by: Jason Ekstrand <<a href="mailto:jason.ekstrand@intel.com">jason.ekstrand@intel.com</a>></p>
<p dir="ltr">> Signed-off-by: Rob Clark <<a href="mailto:robclark@freedesktop.org">robclark@freedesktop.org</a>><br>
> ---<br>
>  src/glsl/nir/nir_lower_idiv.c | 2 --<br>
>  1 file changed, 2 deletions(-)<br>
><br>
> diff --git a/src/glsl/nir/nir_lower_idiv.c b/src/glsl/nir/nir_lower_idiv.c<br>
> index c2f08df..7b68032 100644<br>
> --- a/src/glsl/nir/nir_lower_idiv.c<br>
> +++ b/src/glsl/nir/nir_lower_idiv.c<br>
> @@ -152,6 +152,4 @@ nir_lower_idiv(nir_shader *shader)<br>
>        if (overload->impl)<br>
>           convert_impl(overload->impl);<br>
>     }<br>
> -<br>
> -   exec_list_make_empty(&shader->system_values);<br>
>  }<br>
> --<br>
> 2.1.0<br>
><br>
> _______________________________________________<br>
> mesa-dev mailing list<br>
> <a href="mailto:mesa-dev@lists.freedesktop.org">mesa-dev@lists.freedesktop.org</a><br>
> <a href="http://lists.freedesktop.org/mailman/listinfo/mesa-dev">http://lists.freedesktop.org/mailman/listinfo/mesa-dev</a><br>
</p>