[Mesa-dev] [PATCH 03/15] gallium/tgsi/ureg: Support local temporary emission.

[Jose Fonseca]

Could you add some comments? It's not obvious what's the difference between local temporaries and temporaries, and what is their purpose.

Jose

----- Original Message -----
> ---
>  src/gallium/auxiliary/tgsi/tgsi_ureg.c |   54
>  ++++++++++++++++++++++++++++----
>  src/gallium/auxiliary/tgsi/tgsi_ureg.h |    3 ++
>  2 files changed, 51 insertions(+), 6 deletions(-)
> 
> diff --git a/src/gallium/auxiliary/tgsi/tgsi_ureg.c
> b/src/gallium/auxiliary/tgsi/tgsi_ureg.c
> index 082fec3..6e01416 100644
> --- a/src/gallium/auxiliary/tgsi/tgsi_ureg.c
> +++ b/src/gallium/auxiliary/tgsi/tgsi_ureg.c
> @@ -152,6 +152,7 @@ struct ureg_program
>     unsigned nr_resources;
>  
>     struct util_bitmask *free_temps;
> +   struct util_bitmask *local_temps;
>     unsigned nr_temps;
>  
>     struct const_decl const_decls;
> @@ -530,11 +531,19 @@ out:
>     return ureg_src_register(TGSI_FILE_CONSTANT, index);
>  }
>  
> -struct ureg_dst ureg_DECL_temporary( struct ureg_program *ureg )
> +static struct ureg_dst alloc_temporary( struct ureg_program *ureg,
> +                                        boolean local )
>  {
> +   unsigned i;
> +
>     /* Look for a released temporary.
>      */
> -   unsigned i = util_bitmask_get_first_index(ureg->free_temps);
> +   for (i = util_bitmask_get_first_index(ureg->free_temps);
> +        i != UTIL_BITMASK_INVALID_INDEX;
> +        i = util_bitmask_get_next_index(ureg->free_temps, i + 1)) {
> +      if (util_bitmask_get(ureg->local_temps, i) == local)
> +         break;
> +   }
>  
>     /* Or allocate a new one.
>      */
> @@ -543,9 +552,22 @@ struct ureg_dst ureg_DECL_temporary( struct
> ureg_program *ureg )
>  
>     util_bitmask_clear(ureg->free_temps, i);
>  
> +   if (local)
> +      util_bitmask_set(ureg->local_temps, i);
> +
>     return ureg_dst_register( TGSI_FILE_TEMPORARY, i );
>  }
>  
> +struct ureg_dst ureg_DECL_temporary( struct ureg_program *ureg )
> +{
> +   return alloc_temporary(ureg, FALSE);
> +}
> +
> +struct ureg_dst ureg_DECL_local_temporary( struct ureg_program *ureg
> )
> +{
> +   return alloc_temporary(ureg, TRUE);
> +}
> +
>  void ureg_release_temporary( struct ureg_program *ureg,
>                               struct ureg_dst tmp )
>  {
> @@ -1239,6 +1261,25 @@ emit_decl_fs(struct ureg_program *ureg,
>  }
>  
>  
> +static void emit_decl( struct ureg_program *ureg,
> +                       unsigned file,
> +                       unsigned index,
> +                       boolean local )
> +{
> +   union tgsi_any_token *out = get_tokens( ureg, DOMAIN_DECL, 2 );
> +
> +   out[0].value = 0;
> +   out[0].decl.Type = TGSI_TOKEN_TYPE_DECLARATION;
> +   out[0].decl.NrTokens = 2;
> +   out[0].decl.File = file;
> +   out[0].decl.UsageMask = TGSI_WRITEMASK_XYZW;
> +   out[0].decl.Local = local;
> +
> +   out[1].value = 0;
> +   out[1].decl_range.First = index;
> +   out[1].decl_range.Last = index;
> +}
> +
>  static void emit_decl_range( struct ureg_program *ureg,
>                               unsigned file,
>                               unsigned first,
> @@ -1493,10 +1534,9 @@ static void emit_decls( struct ureg_program
> *ureg )
>        }
>     }
>  
> -   if (ureg->nr_temps) {
> -      emit_decl_range( ureg,
> -                       TGSI_FILE_TEMPORARY,
> -                       0, ureg->nr_temps );
> +   for (i = 0; i < ureg->nr_temps; i++) {
> +      emit_decl( ureg, TGSI_FILE_TEMPORARY, i,
> +                 util_bitmask_get(ureg->local_temps, i) );
>     }
>  
>     if (ureg->nr_addrs) {
> @@ -1655,6 +1695,7 @@ struct ureg_program *ureg_create( unsigned
> processor )
>     ureg->property_gs_max_vertices = ~0;
>  
>     ureg->free_temps = util_bitmask_create();
> +   ureg->local_temps = util_bitmask_create();
>  
>     return ureg;
>  }
> @@ -1671,6 +1712,7 @@ void ureg_destroy( struct ureg_program *ureg )
>     }
>  
>     util_bitmask_destroy(ureg->free_temps);
> +   util_bitmask_destroy(ureg->local_temps);
>  
>     FREE(ureg);
>  }
> diff --git a/src/gallium/auxiliary/tgsi/tgsi_ureg.h
> b/src/gallium/auxiliary/tgsi/tgsi_ureg.h
> index 07ab8cb..5ec06a8 100644
> --- a/src/gallium/auxiliary/tgsi/tgsi_ureg.h
> +++ b/src/gallium/auxiliary/tgsi/tgsi_ureg.h
> @@ -272,6 +272,9 @@ ureg_DECL_constant( struct ureg_program *,
>  struct ureg_dst
>  ureg_DECL_temporary( struct ureg_program * );
>  
> +struct ureg_dst
> +ureg_DECL_local_temporary( struct ureg_program * );
> +
>  void
>  ureg_release_temporary( struct ureg_program *ureg,
>                          struct ureg_dst tmp );
> --
> 1.7.9.2
> 
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev
>