[Mesa-dev] [PATCH 0/7] release build warning fixes

[Ian Romanick]

I've put this series with the collected R-b on the maybe_unused branch
of my fd.o Mesa repo.  I'll wait a bit, rebase the reviewed patches
first, then push the reviewed patches.  I don't think there's much point
in sitting on the reviewed patches for a long time.  Right?

On 04/15/2016 06:00 PM, Grazvydas Ignotas wrote:
> I guess almost everyone here builds mesa with asserts enabled, but many
> forget asserts are off by default, so ordinary users get thrown at quite
> a lot of warnings. This tries to improve the situation somewhat.
> 
> Grazvydas Ignotas (7):
>   util: add MAYBE_UNUSED for config dependent variables
>   glsl: fix warning in release build
>   gallium: fix warnings in release build
>   spirv: fix warning in release build
>   isl: fix warnings in release build
>   anv: fix warnings in release build
>   gallium: use unreachable instead of asserts
> 
>  src/compiler/glsl/lower_buffer_access.cpp           | 2 +-
>  src/compiler/spirv/vtn_cfg.c                        | 2 +-
>  src/gallium/auxiliary/gallivm/lp_bld_intr.c         | 2 +-
>  src/gallium/auxiliary/pipebuffer/pb_buffer_fenced.c | 2 +-
>  src/gallium/auxiliary/util/u_surface.c              | 3 ++-
>  src/gallium/drivers/llvmpipe/lp_setup_vbuf.c        | 2 +-
>  src/gallium/drivers/r600/compute_memory_pool.c      | 2 +-
>  src/gallium/drivers/radeon/radeon_llvm_emit.c       | 2 +-
>  src/gallium/drivers/softpipe/sp_state_sampler.c     | 2 +-
>  src/gallium/drivers/softpipe/sp_tile_cache.c        | 3 ++-
>  src/gallium/drivers/trace/tr_context.c              | 2 +-
>  src/intel/isl/isl.c                                 | 2 +-
>  src/intel/isl/isl_gen7.c                            | 2 +-
>  src/intel/vulkan/anv_dump.c                         | 2 +-
>  src/intel/vulkan/anv_image.c                        | 2 +-
>  src/intel/vulkan/anv_nir_lower_push_constants.c     | 2 +-
>  src/intel/vulkan/gen7_cmd_buffer.c                  | 2 +-
>  src/intel/vulkan/gen8_cmd_buffer.c                  | 2 +-
>  src/util/macros.h                                   | 2 ++
>  19 files changed, 22 insertions(+), 18 deletions(-)
>