LGTM

Reviewed-by: Jason Ekstrand <jason.ekstr...@intel.com>

On Mon, Mar 9, 2015 at 1:58 AM, Kenneth Graunke <kenn...@whitecape.org>
wrote:

> Array variable copy splitting generates a bunch of stuff we want to
> clean up before proceeding.
>
> Signed-off-by: Kenneth Graunke <kenn...@whitecape.org>
> Cc: Jason Ekstrand <ja...@jlekstrand.net>
> ---
>  src/mesa/drivers/dri/i965/brw_fs_nir.cpp | 3 +++
>  1 file changed, 3 insertions(+)
>
> diff --git a/src/mesa/drivers/dri/i965/brw_fs_nir.cpp
> b/src/mesa/drivers/dri/i965/brw_fs_nir.cpp
> index 249e59c..fbdfc22 100644
> --- a/src/mesa/drivers/dri/i965/brw_fs_nir.cpp
> +++ b/src/mesa/drivers/dri/i965/brw_fs_nir.cpp
> @@ -102,6 +102,9 @@ fs_visitor::emit_nir_code()
>     nir_lower_var_copies(nir);
>     nir_validate_shader(nir);
>
> +   /* Get rid of split copies */
> +   nir_optimize(nir);
> +
>     nir_lower_io(nir);
>     nir_validate_shader(nir);
>
> --
> 2.2.1
>
>
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to