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(-) > _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev
