[Mesa-dev] [PATCH 2/9] i965/nir: Optimize after nir_lower_var_copies().

Jason Ekstrand jason at jlekstrand.net
Mon Mar 9 11:12:02 PDT 2015


LGTM

Reviewed-by: Jason Ekstrand <jason.ekstrand at intel.com>

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

> Array variable copy splitting generates a bunch of stuff we want to
> clean up before proceeding.
>
> Signed-off-by: Kenneth Graunke <kenneth at whitecape.org>
> Cc: Jason Ekstrand <jason at 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
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/mesa-dev/attachments/20150309/966ae839/attachment-0001.html>


More information about the mesa-dev mailing list