[Mesa-dev] [PATCH v2] i965: fix release build unused variable warning

Eduardo Lima Mitev elima at igalia.com
Mon Dec 12 06:14:16 UTC 2016


On 12/11/2016 08:57 PM, Grazvydas Ignotas wrote:
> Signed-off-by: Grazvydas Ignotas <notasas at gmail.com>
> ---
> v2: take Eduardo Lima's suggestion further - remove the variable completely
> no commit access
> 

I just pushed the patch. Thanks!

Eduardo

>  src/mesa/drivers/dri/i965/brw_blorp.c | 4 +---
>  1 file changed, 1 insertion(+), 3 deletions(-)
> 
> diff --git a/src/mesa/drivers/dri/i965/brw_blorp.c b/src/mesa/drivers/dri/i965/brw_blorp.c
> index 4c1d858..43ac3be 100644
> --- a/src/mesa/drivers/dri/i965/brw_blorp.c
> +++ b/src/mesa/drivers/dri/i965/brw_blorp.c
> @@ -813,8 +813,6 @@ do_single_blorp_clear(struct brw_context *brw, struct gl_framebuffer *fb,
>        can_fast_clear = false;
>  
>     const unsigned logical_layer = irb_logical_mt_layer(irb);
> -   const bool is_lossless_compressed = intel_miptree_is_lossless_compressed(
> -                                          brw, irb->mt);
>     const enum intel_fast_clear_state fast_clear_state =
>        intel_miptree_get_fast_clear_state(irb->mt, irb->mt_level,
>                                           logical_layer);
> @@ -850,7 +848,7 @@ do_single_blorp_clear(struct brw_context *brw, struct gl_framebuffer *fb,
>         * it now.
>         */
>        if (!irb->mt->mcs_buf) {
> -         assert(!is_lossless_compressed);
> +         assert(!intel_miptree_is_lossless_compressed(brw, irb->mt));
>           if (!intel_miptree_alloc_non_msrt_mcs(brw, irb->mt, false)) {
>              /* MCS allocation failed--probably this will only happen in
>               * out-of-memory conditions.  But in any case, try to recover
> 



More information about the mesa-dev mailing list