From 349fe79c9bc9bd20b877f0425763509208179b47 Mon Sep 17 00:00:00 2001 From: Kenneth Graunke Date: Tue, 12 Jul 2016 02:16:30 -0700 Subject: [PATCH] nir: Share get_io_offset handling in nir_lower_io. The load/store/atomic cases all duplicated the get_io_offset code, with a few tiny differences: stores didn't bother checking for per-vertex inputs, because they can't be stored to, and atomics didn't check at all, since shared variables aren't per-vertex. However, it's harmless to check, and allows us to share more code. Signed-off-by: Kenneth Graunke Reviewed-by: Jason Ekstrand --- src/compiler/nir/nir_lower_io.c | 33 +++++++++------------------------ 1 file changed, 9 insertions(+), 24 deletions(-) diff --git a/src/compiler/nir/nir_lower_io.c b/src/compiler/nir/nir_lower_io.c index 9b53eff..45cc671 100644 --- a/src/compiler/nir/nir_lower_io.c +++ b/src/compiler/nir/nir_lower_io.c @@ -279,19 +279,18 @@ nir_lower_io_block(nir_block *block, b->cursor = nir_before_instr(instr); - switch (intrin->intrinsic) { - case nir_intrinsic_load_var: { - const bool per_vertex = - is_per_vertex_input(state, var) || - is_per_vertex_output(state, var); + const bool per_vertex = + is_per_vertex_input(state, var) || is_per_vertex_output(state, var); - nir_ssa_def *offset; - nir_ssa_def *vertex_index; + nir_ssa_def *offset; + nir_ssa_def *vertex_index; - offset = get_io_offset(b, intrin->variables[0], - per_vertex ? &vertex_index : NULL, - state->type_size); + offset = get_io_offset(b, intrin->variables[0], + per_vertex ? &vertex_index : NULL, + state->type_size); + switch (intrin->intrinsic) { + case nir_intrinsic_load_var: { nir_intrinsic_instr *load = nir_intrinsic_instr_create(state->mem_ctx, load_op(mode, per_vertex)); @@ -330,15 +329,6 @@ nir_lower_io_block(nir_block *block, case nir_intrinsic_store_var: { assert(mode == nir_var_shader_out || mode == nir_var_shared); - nir_ssa_def *offset; - nir_ssa_def *vertex_index; - - const bool per_vertex = is_per_vertex_output(state, var); - - offset = get_io_offset(b, intrin->variables[0], - per_vertex ? &vertex_index : NULL, - state->type_size); - nir_intrinsic_instr *store = nir_intrinsic_instr_create(state->mem_ctx, store_op(state, mode, per_vertex)); @@ -375,11 +365,6 @@ nir_lower_io_block(nir_block *block, case nir_intrinsic_var_atomic_comp_swap: { assert(mode == nir_var_shared); - nir_ssa_def *offset; - - offset = get_io_offset(b, intrin->variables[0], - NULL, state->type_size); - nir_intrinsic_instr *atomic = nir_intrinsic_instr_create(state->mem_ctx, atomic_op(intrin->intrinsic)); -- 2.7.4