[Mesa-dev] [PATCH 087/133] i965/fs_nir: Don't dump the shader.

Connor Abbott cwabbott0 at gmail.com
Tue Dec 16 13:41:34 PST 2014


Can we squash this into wherever we introduced these print statements,
and instead port over Matt's i965 fs debugging infrastructure so we
can do this with an envvar enable, plus all the other goodies?

On Tue, Dec 16, 2014 at 1:11 AM, Jason Ekstrand <jason at jlekstrand.net> wrote:
> This is killing piglit.  I'll leave the logging local
> ---
>  src/mesa/drivers/dri/i965/brw_fs_nir.cpp | 5 -----
>  1 file changed, 5 deletions(-)
>
> diff --git a/src/mesa/drivers/dri/i965/brw_fs_nir.cpp b/src/mesa/drivers/dri/i965/brw_fs_nir.cpp
> index f4656bc..0cd8fca 100644
> --- a/src/mesa/drivers/dri/i965/brw_fs_nir.cpp
> +++ b/src/mesa/drivers/dri/i965/brw_fs_nir.cpp
> @@ -51,9 +51,6 @@ fs_visitor::emit_nir_code()
>     nir_opt_global_to_local(nir);
>     nir_validate_shader(nir);
>
> -   if (1)
> -      nir_print_shader(nir, stderr);
> -
>     nir_convert_to_ssa(nir);
>     nir_validate_shader(nir);
>
> @@ -71,11 +68,9 @@ fs_visitor::emit_nir_code()
>        progress |= nir_opt_peephole_ffma(nir);
>        nir_validate_shader(nir);
>     } while (progress);
> -   nir_print_shader(nir, stderr);
>
>     nir_convert_from_ssa(nir);
>     nir_validate_shader(nir);
> -   nir_print_shader(nir, stderr);
>     nir_lower_vec_to_movs(nir);
>     nir_validate_shader(nir);
>
> --
> 2.2.0
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev


More information about the mesa-dev mailing list