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

2016-12-11 Thread Eduardo Lima Mitev
The patch looks good but now that we are at it, I would move the definition and assignment of 'is_lossless_compressed' to the block where it is used, right before the assert. Right now we are always defining and assigning the variable, but it is consumed conditionally. With that changed, patch is:

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

2016-12-10 Thread Grazvydas Ignotas
Signed-off-by: Grazvydas Ignotas --- src/mesa/drivers/dri/i965/brw_blorp.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/mesa/drivers/dri/i965/brw_blorp.c b/src/mesa/drivers/dri/i965/brw_blorp.c index 4c1d858..8b5750c 100644 --- a/src/mesa/drivers/dri/i965/brw_blorp