diff options
author | Ian Romanick <[email protected]> | 2015-08-25 18:19:12 +0100 |
---|---|---|
committer | Ian Romanick <[email protected]> | 2016-05-12 16:46:41 -0700 |
commit | 098166e1bc68860b2be9230e5241ead014bf254c (patch) | |
tree | 5dc9028b3637cac4ec0debf70e990cc34efecba9 /src | |
parent | bd9ae724591d23d029ee9f8a2f200b60bf6fba2c (diff) |
nir: Silence unused parameter warnings
These cases had the parameter removed:
nir/nir_lower_vec_to_movs.c: In function ‘try_coalesce’:
nir/nir_lower_vec_to_movs.c:124:66: warning: unused parameter ‘shader’ [-Wunused-parameter]
try_coalesce(nir_alu_instr *vec, unsigned start_idx, nir_shader *shader)
^
nir/nir_lower_io.c: In function ‘load_op’:
nir/nir_lower_io.c:147:32: warning: unused parameter ‘state’ [-Wunused-parameter]
load_op(struct lower_io_state *state,
^
These cases had the parameter (void) silenced because the parameter was
necessary for an interface:
nir/glsl_to_nir.cpp:1900:32: warning: unused parameter 'ir' [-Wunused-parameter]
nir_visitor::visit(ir_barrier *ir)
^
nir/nir.c: In function ‘remove_use_cb’:
nir/nir.c:802:35: warning: unused parameter ‘state’ [-Wunused-parameter]
remove_use_cb(nir_src *src, void *state)
^
nir/nir.c: In function ‘remove_def_cb’:
nir/nir.c:811:37: warning: unused parameter ‘state’ [-Wunused-parameter]
remove_def_cb(nir_dest *dest, void *state)
^
Number of total warnings in my build reduced from 2543 to 2538
(reduction of 5).
Signed-off-by: Ian Romanick <[email protected]>
Reviewed-by: Kenneth Graunke <[email protected]>
Diffstat (limited to 'src')
-rw-r--r-- | src/compiler/nir/glsl_to_nir.cpp | 2 | ||||
-rw-r--r-- | src/compiler/nir/nir.c | 4 | ||||
-rw-r--r-- | src/compiler/nir/nir_lower_io.c | 5 | ||||
-rw-r--r-- | src/compiler/nir/nir_lower_vec_to_movs.c | 4 |
4 files changed, 9 insertions, 6 deletions
diff --git a/src/compiler/nir/glsl_to_nir.cpp b/src/compiler/nir/glsl_to_nir.cpp index aeeba5d5573..fb1d4218e33 100644 --- a/src/compiler/nir/glsl_to_nir.cpp +++ b/src/compiler/nir/glsl_to_nir.cpp @@ -2007,7 +2007,7 @@ nir_visitor::visit(ir_dereference_array *ir) } void -nir_visitor::visit(ir_barrier *ir) +nir_visitor::visit(ir_barrier *) { nir_intrinsic_instr *instr = nir_intrinsic_instr_create(this->shader, nir_intrinsic_barrier); diff --git a/src/compiler/nir/nir.c b/src/compiler/nir/nir.c index 71adcb36267..cf6ca522c8b 100644 --- a/src/compiler/nir/nir.c +++ b/src/compiler/nir/nir.c @@ -893,6 +893,8 @@ src_is_valid(const nir_src *src) static bool remove_use_cb(nir_src *src, void *state) { + (void) state; + if (src_is_valid(src)) list_del(&src->use_link); @@ -902,6 +904,8 @@ remove_use_cb(nir_src *src, void *state) static bool remove_def_cb(nir_dest *dest, void *state) { + (void) state; + if (!dest->is_ssa) list_del(&dest->reg.def_link); diff --git a/src/compiler/nir/nir_lower_io.c b/src/compiler/nir/nir_lower_io.c index d0f3dc72884..a8399245b0b 100644 --- a/src/compiler/nir/nir_lower_io.c +++ b/src/compiler/nir/nir_lower_io.c @@ -144,8 +144,7 @@ get_io_offset(nir_builder *b, nir_deref_var *deref, } static nir_intrinsic_op -load_op(struct lower_io_state *state, - nir_variable_mode mode, bool per_vertex) +load_op(nir_variable_mode mode, bool per_vertex) { nir_intrinsic_op op; switch (mode) { @@ -270,7 +269,7 @@ nir_lower_io_block(nir_block *block, nir_intrinsic_instr *load = nir_intrinsic_instr_create(state->mem_ctx, - load_op(state, mode, per_vertex)); + load_op(mode, per_vertex)); load->num_components = intrin->num_components; nir_intrinsic_set_base(load, diff --git a/src/compiler/nir/nir_lower_vec_to_movs.c b/src/compiler/nir/nir_lower_vec_to_movs.c index b383fb8ccf2..711ddd38bda 100644 --- a/src/compiler/nir/nir_lower_vec_to_movs.c +++ b/src/compiler/nir/nir_lower_vec_to_movs.c @@ -118,7 +118,7 @@ has_replicated_dest(nir_alu_instr *alu) * can then call insert_mov as normal. */ static unsigned -try_coalesce(nir_alu_instr *vec, unsigned start_idx, nir_shader *shader) +try_coalesce(nir_alu_instr *vec, unsigned start_idx) { assert(start_idx < nir_op_infos[vec->op].num_inputs); @@ -264,7 +264,7 @@ lower_vec_to_movs_block(nir_block *block, nir_function_impl *impl) continue; if (!(finished_write_mask & (1 << i))) - finished_write_mask |= try_coalesce(vec, i, shader); + finished_write_mask |= try_coalesce(vec, i); if (!(finished_write_mask & (1 << i))) finished_write_mask |= insert_mov(vec, i, shader); |