[Mesa-dev] [PATCH 2/2] nir: Remove unused variable unroll_loc

Timothy Arceri tarceri at itsqueeze.com
Tue Mar 12 03:06:01 UTC 2019



On 12/3/19 1:30 pm, Alyssa Rosenzweig wrote:
> Fixes a gcc warning.
> 
> Signed-off-by: Alyssa Rosenzweig <alyssa at rosenzweig.io>
> ---
>   src/compiler/nir/nir_opt_loop_unroll.c | 4 ----
>   1 file changed, 4 deletions(-)
> 
> diff --git a/src/compiler/nir/nir_opt_loop_unroll.c b/src/compiler/nir/nir_opt_loop_unroll.c
> index 9ab0a924c82..41f7a834164 100644
> --- a/src/compiler/nir/nir_opt_loop_unroll.c
> +++ b/src/compiler/nir/nir_opt_loop_unroll.c
> @@ -488,12 +488,8 @@ complex_unroll_single_terminator(nir_loop *loop)
>       * vars for the last iteration (they are inside the following ifs break
>       * branch). We leave other passes to clean up this redundant if.
>       */
> -   unsigned num_times_to_clone = loop->info->max_trip_count + 1;
>   
>      nir_cf_list lp_body;
> -   nir_cf_node *unroll_loc =
> -      complex_unroll_loop_body(loop, terminator, &lp_header, &lp_body,
> -                               remap_table, num_times_to_clone);

A better fix might be to delete the Mesa src tree, that should fix up 
any warnings :P

I can only assume this was sent out by mistake?

>   
>      /* Delete the original loop header and body */
>      nir_cf_delete(&lp_header);
> 


More information about the mesa-dev mailing list