<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Tue, Dec 16, 2014 at 1:41 PM, Connor Abbott <span dir="ltr"><<a href="mailto:cwabbott0@gmail.com" target="_blank">cwabbott0@gmail.com</a>></span> wrote:<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Can we squash this into wherever we introduced these print statements,<br>
and instead port over Matt's i965 fs debugging infrastructure so we<br>
can do this with an envvar enable, plus all the other goodies?<br></blockquote><div><br></div><div>No, there isn't a single patch I can squish this with.  Yes, I could unwind shader printing from the series and just remove the places where we print it.  No, I'd rather not (see also the note in the 000 about rebasing, history, etc.  also, It's more work than just a squash).<br><br></div><div>RE: Matt's nifty macro.  Absolutely. I'm planning to do that, but it should be easy enough to do as its own patch later.<br></div><div>--Jason<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div><div class="h5"><br>
On Tue, Dec 16, 2014 at 1:11 AM, Jason Ekstrand <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>> wrote:<br>
> This is killing piglit.  I'll leave the logging local<br>
> ---<br>
>  src/mesa/drivers/dri/i965/brw_fs_nir.cpp | 5 -----<br>
>  1 file changed, 5 deletions(-)<br>
><br>
> diff --git a/src/mesa/drivers/dri/i965/brw_fs_nir.cpp b/src/mesa/drivers/dri/i965/brw_fs_nir.cpp<br>
> index f4656bc..0cd8fca 100644<br>
> --- a/src/mesa/drivers/dri/i965/brw_fs_nir.cpp<br>
> +++ b/src/mesa/drivers/dri/i965/brw_fs_nir.cpp<br>
> @@ -51,9 +51,6 @@ fs_visitor::emit_nir_code()<br>
>     nir_opt_global_to_local(nir);<br>
>     nir_validate_shader(nir);<br>
><br>
> -   if (1)<br>
> -      nir_print_shader(nir, stderr);<br>
> -<br>
>     nir_convert_to_ssa(nir);<br>
>     nir_validate_shader(nir);<br>
><br>
> @@ -71,11 +68,9 @@ fs_visitor::emit_nir_code()<br>
>        progress |= nir_opt_peephole_ffma(nir);<br>
>        nir_validate_shader(nir);<br>
>     } while (progress);<br>
> -   nir_print_shader(nir, stderr);<br>
><br>
>     nir_convert_from_ssa(nir);<br>
>     nir_validate_shader(nir);<br>
> -   nir_print_shader(nir, stderr);<br>
>     nir_lower_vec_to_movs(nir);<br>
>     nir_validate_shader(nir);<br>
><br>
> --<br>
> 2.2.0<br>
><br>
</div></div>> _______________________________________________<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" target="_blank">http://lists.freedesktop.org/mailman/listinfo/mesa-dev</a><br>
</blockquote></div></div></div>