diff options
author | Kenneth Graunke <[email protected]> | 2016-04-20 01:15:53 -0700 |
---|---|---|
committer | Kenneth Graunke <[email protected]> | 2016-05-05 14:24:00 -0700 |
commit | 49ea7454a189d9ebfa03fddfdba9949c6eaf4caa (patch) | |
tree | 3f0dc558592fdf903c937d0dadbb164254761399 /src/compiler/nir | |
parent | a808ba59657b3e5c6399e51fa1f4ebe9cad201a9 (diff) |
nir: Rename opt_undef_alu to opt_undef_csel; update comments.
This better reflects what it does. I plan to add other ALU
optimizations as well, so the old name would be confusing.
In preparation for that, also move the file comments about csels
above the opt_undef_csel function, and delete the ones about there
not being other optimizations.
Signed-off-by: Kenneth Graunke <[email protected]>
Reviewed-by: Matt Turner <[email protected]>
Diffstat (limited to 'src/compiler/nir')
-rw-r--r-- | src/compiler/nir/nir_opt_undef.c | 25 |
1 files changed, 13 insertions, 12 deletions
diff --git a/src/compiler/nir/nir_opt_undef.c b/src/compiler/nir/nir_opt_undef.c index 14459ff62af..ce18c2df043 100644 --- a/src/compiler/nir/nir_opt_undef.c +++ b/src/compiler/nir/nir_opt_undef.c @@ -25,18 +25,17 @@ /** @file nir_opt_undef.c * - * Handles optimization of operations involving ssa_undef. For now, we just - * make sure that csels between undef and some other value just give the other - * value (on the assumption that the condition's going to be choosing the - * defined value). This reduces work after if flattening when each side of - * the if is defining a variable. - * - * Some day, we may find some use for making other operations consuming an - * undef arg output undef, but I don't know of any cases currently. + * Handles optimization of operations involving ssa_undef. */ +/** + * Turn conditional selects between an undef and some other value into a move + * of that other value (on the assumption that the condition's going to be + * choosing the defined value). This reduces work after if flattening when + * each side of the if is defining a variable. + */ static bool -opt_undef_alu(nir_alu_instr *instr) +opt_undef_csel(nir_alu_instr *instr) { if (instr->op != nir_op_bcsel && instr->op != nir_op_fcsel) return false; @@ -80,9 +79,11 @@ nir_opt_undef(nir_shader *shader) if (function->impl) { nir_foreach_block(block, function->impl) { nir_foreach_instr_safe(instr, block) { - if (instr->type == nir_instr_type_alu) - if (opt_undef_alu(nir_instr_as_alu(instr))) - progress = true; + if (instr->type == nir_instr_type_alu) { + nir_alu_instr *alu = nir_instr_as_alu(instr); + + progress = opt_undef_csel(alu) || progress; + } } } |