LGTM

Reviewed-by: Jason Ekstrand <[email protected]>

On Mon, Mar 9, 2015 at 1:58 AM, Kenneth Graunke <[email protected]>
wrote:

> Array variable copy splitting generates a bunch of stuff we want to
> clean up before proceeding.
>
> Signed-off-by: Kenneth Graunke <[email protected]>
> Cc: Jason Ekstrand <[email protected]>
> ---
>  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
[email protected]
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to