[Mesa-dev] [PATCH] nir: rename variables in nir_lower_io_to_temporaries for clarity

[Samuel Iglesias Gonsálvez]

Reviewed-by: Samuel Iglesias Gonsálvez <siglesias at igalia.com>

Sam

On 05/04/18 01:16, Caio Marcelo de Oliveira Filho wrote:
> In the emit_copies() function, the use of "newv" and "temp" names made
> sense when only copies from temporaries to the new variables were
> being done. But now there are other calls to copy with other pairings,
> and "temp" doesn't always refer to a temporary created in this
> pass. Use the names "dest" and "src" instead.
> ---
>  .../nir/nir_lower_io_to_temporaries.c         | 22 +++++++++----------
>  1 file changed, 11 insertions(+), 11 deletions(-)
>
> diff --git a/src/compiler/nir/nir_lower_io_to_temporaries.c b/src/compiler/nir/nir_lower_io_to_temporaries.c
> index 301ba65892..c3e1207f4e 100644
> --- a/src/compiler/nir/nir_lower_io_to_temporaries.c
> +++ b/src/compiler/nir/nir_lower_io_to_temporaries.c
> @@ -40,34 +40,34 @@ struct lower_io_state {
>  };
>  
>  static void
> -emit_copies(nir_cursor cursor, nir_shader *shader, struct exec_list *new_vars,
> -          struct exec_list *old_vars)
> +emit_copies(nir_cursor cursor, nir_shader *shader, struct exec_list *dest_vars,
> +            struct exec_list *src_vars)
>  {
> -   assert(exec_list_length(new_vars) == exec_list_length(old_vars));
> +   assert(exec_list_length(dest_vars) == exec_list_length(src_vars));
>  
> -   foreach_two_lists(new_node, new_vars, old_node, old_vars) {
> -      nir_variable *newv = exec_node_data(nir_variable, new_node, node);
> -      nir_variable *temp = exec_node_data(nir_variable, old_node, node);
> +   foreach_two_lists(new_node, dest_vars, old_node, src_vars) {
> +      nir_variable *dest = exec_node_data(nir_variable, new_node, node);
> +      nir_variable *src = exec_node_data(nir_variable, old_node, node);
>  
>        /* No need to copy the contents of a non-fb_fetch_output output variable
>         * to the temporary allocated for it, since its initial value is
>         * undefined.
>         */
> -      if (temp->data.mode == nir_var_shader_out &&
> -          !temp->data.fb_fetch_output)
> +      if (src->data.mode == nir_var_shader_out &&
> +          !src->data.fb_fetch_output)
>           continue;
>  
>        /* Can't copy the contents of the temporary back to a read-only
>         * interface variable.  The value of the temporary won't have been
>         * modified by the shader anyway.
>         */
> -      if (newv->data.read_only)
> +      if (dest->data.read_only)
>           continue;
>  
>        nir_intrinsic_instr *copy =
>           nir_intrinsic_instr_create(shader, nir_intrinsic_copy_var);
> -      copy->variables[0] = nir_deref_var_create(copy, newv);
> -      copy->variables[1] = nir_deref_var_create(copy, temp);
> +      copy->variables[0] = nir_deref_var_create(copy, dest);
> +      copy->variables[1] = nir_deref_var_create(copy, src);
>  
>        nir_instr_insert(cursor, &copy->instr);
>     }

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20180406/f349a15b/attachment-0001.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 801 bytes
Desc: OpenPGP digital signature
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20180406/f349a15b/attachment-0001.sig>