Thanks Ken, this does make debugging NIR code a bit easier :)
Reviewed-by: Iago Toral Quiroga <ito...@igalia.com>
On Wed, 2015-06-10 at 02:39 -0700, Kenneth Graunke wrote:
> This makes the SSA definitions use sequential numbers (0, 1, 2, ...)
> instead of seemingly random ones. There's not much point normally,
> but it makes debug output much easier to read.
>
> Signed-off-by: Kenneth Graunke <kenn...@whitecape.org>
> ---
> src/mesa/drivers/dri/i965/brw_nir.c | 6 ++++++
> 1 file changed, 6 insertions(+)
>
> diff --git a/src/mesa/drivers/dri/i965/brw_nir.c
> b/src/mesa/drivers/dri/i965/brw_nir.c
> index 142162c..c13708a 100644
> --- a/src/mesa/drivers/dri/i965/brw_nir.c
> +++ b/src/mesa/drivers/dri/i965/brw_nir.c
> @@ -167,6 +167,12 @@ brw_create_nir(struct brw_context *brw,
> nir_validate_shader(nir);
>
> if (unlikely(debug_enabled)) {
> + /* Re-index SSA defs so we print more sensible numbers. */
> + nir_foreach_overload(nir, overload) {
> + if (overload->impl)
> + nir_index_ssa_defs(overload->impl);
> + }
> +
> fprintf(stderr, "NIR (SSA form) for %s shader:\n",
> _mesa_shader_stage_to_string(stage));
> nir_print_shader(nir, stderr);
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
