Since we handle booleans as integers this makes more sense. v2: - rebased to incorporate new boolean conversion opcodes
Reviewed-by: Topi Pohjolainen <topi.pohjolai...@intel.com> (v1) Reviewed-by: Jason Ekstrand <ja...@jlekstrand.net> (v1) --- src/intel/compiler/brw_fs_nir.cpp | 20 ++++++++++---------- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/src/intel/compiler/brw_fs_nir.cpp b/src/intel/compiler/brw_fs_nir.cpp index dfce2441fef..d12b5bc7652 100644 --- a/src/intel/compiler/brw_fs_nir.cpp +++ b/src/intel/compiler/brw_fs_nir.cpp @@ -820,16 +820,6 @@ fs_visitor::nir_emit_alu(const fs_builder &bld, nir_alu_instr *instr) inst->saturate = instr->dest.saturate; break; - case nir_op_b2i8: - case nir_op_b2i16: - case nir_op_b2i32: - case nir_op_b2i64: - case nir_op_b2f16: - case nir_op_b2f32: - case nir_op_b2f64: - op[0].type = BRW_REGISTER_TYPE_D; - op[0].negate = !op[0].negate; - /* fallthrough */ case nir_op_f2f64: case nir_op_f2i64: case nir_op_f2u64: @@ -840,6 +830,16 @@ fs_visitor::nir_emit_alu(const fs_builder &bld, nir_alu_instr *instr) inst->saturate = instr->dest.saturate; break; + case nir_op_b2i8: + case nir_op_b2i16: + case nir_op_b2i32: + case nir_op_b2i64: + case nir_op_b2f16: + case nir_op_b2f32: + case nir_op_b2f64: + op[0].type = BRW_REGISTER_TYPE_D; + op[0].negate = !op[0].negate; + /* fallthrough */ case nir_op_i2f64: case nir_op_i2i64: case nir_op_u2f64: -- 2.17.1 _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev